User tests: Successful: Unsuccessful:
This Backports the fix from the framework into the CMS:
joomla-framework/keychain@7e6488d
Thanks goes to @photodude here: #6672
Labels |
Added:
?
|
Labels |
Added:
?
|
Category | ⇒ | External Library |
Status | New | ⇒ | Pending |
@Kubik-Rubik you need to look at the method you are using to apply this. There is only one file changed in this PR and that file has not been changed for 4 months. The merge issue is due to the way YOU are doing it. There is nothing wrong with the PR or any need for it to be updated
Merge on review (and since it's the same fix applied in the Framework repo)
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-05-09 15:40:07 |
Closed_By | ⇒ | mbabker |
@brianteeman I have tried to merge the same branch and I get the same errors. I am not doing this in any way different than the other PRs. Why should I? Perhaps I am doing it wrong as well and don't know how to do it. There are other ways to merge but we are trying to keep our log clean.
Well @mbabker managed to do it :)
On 9 May 2015 4:54 pm, "RolandD" notifications@github.com wrote:
@brianteeman https://github.com/brianteeman I have tried to merge the
same branch and I get the same errors. I am not doing this in any way
different than the other PRs. Why should I? Perhaps I am doing it wrong as
well and don't know how to do it. There are other ways to merge but we aretrying to keep our log clean.
This comment was created with the J!Tracker Application
https://github.com/joomla/jissues at issues.joomla.org/joomla-cms/6883
http://issues.joomla.org/tracker/joomla-cms/6883.—
Reply to this email directly or view it on GitHub
#6883 (comment).
@roland-d @Kubik-Rubik hmm i guess the problem is that i have 2 accounts (zero-24 and zero24). One with push access and one without push access
This branche is 300+ commits behind staging
https://github.com/zero-24/joomla-cms/tree/patch-5
But this (is the branche of the patch)
https://github.com/zero24/joomla-cms/tree/patch-5
is ok. Sorry that this is confused.
Milestone |
Added: |
That shouldnt make any difference. It is not reasonable to expect all pull
requests to always be up to date with current staging.
On 9 May 2015 18:56, "zero-24" notifications@github.com wrote:
@roland-d https://github.com/roland-d @Kubik-Rubik
https://github.com/Kubik-Rubik hmm i guess the problem is that i have 2
accounts (zero-24 and zero24). One with push access and one without push
access [image: ]This branche is 300+ commits behind staging
https://github.com/zero-24/joomla-cms/tree/patch-5But this (is the branche of the patch)
https://github.com/zero24/joomla-cms/tree/patch-5is ok. Sorry that this is confused.
—
Reply to this email directly or view it on GitHub
#6883 (comment).
@zero-24 Yes, this was the problem. We used the account that you use actively here... The "zero24" branch works without a problem.
@brianteeman Haha, yes, push the green button! :-D By the way, you can not know the way how we in the PLT do the commits properly (no, not the merge button at GitHub), so your comments are acceptable!
Just remember there are a few people not on PLT who may review/merge patches that should be kept in the loop on policy decisions...
Not always necessary, especially when the PR is only a single commit (like this one), unless you're trying to avoid the "merged PR whatever" commits, then someone decided something and forgot to tell me.
As @mbabker said you need to make sure ALL people with commit rights know
On 9 May 2015 21:49, "Viktor Vogel" notifications@github.com wrote:
@mbabker https://github.com/mbabker This is something we want to
discuss at the PLT summit after JAB and establish a policy rule.—
Reply to this email directly or view it on GitHub
#6883 (comment).
@zero-24 Thanks for the acknowledgment. Glad to see this fix going into place.