? Pending

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
12 Jan 2017
avatar wilsonge wilsonge - open - 12 Jan 2017
avatar wilsonge wilsonge - change - 12 Jan 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 12 Jan 2017
Category Repository Administration com_admin SQL Postgresql
avatar jeckodevelopment jeckodevelopment - change - 12 Jan 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-01-12 12:26:23
Closed_By jeckodevelopment
Labels Added: ?
avatar jeckodevelopment jeckodevelopment - close - 12 Jan 2017
avatar jeckodevelopment jeckodevelopment - merge - 12 Jan 2017
avatar andrepereiradasilva
andrepereiradasilva - comment - 12 Jan 2017

actually bump.php would do this automatticly ? on next run
https://github.com/joomla/joomla-cms/blob/staging/build/bump.php#L307

avatar dgt41
dgt41 - comment - 12 Jan 2017

@andrepereiradasilva you should have mentioned that a bit earlier, now almost all the PRs are out of sync... ?

avatar andrepereiradasilva
andrepereiradasilva - comment - 12 Jan 2017

i mention it earlier ... back in june #10865 ?

avatar dgt41
dgt41 - comment - 12 Jan 2017

So @wilsonge's fault then ?

avatar mbabker
mbabker - comment - 12 Jan 2017

now almost all the PRs are out of sync...

Changing the file header doesn't make PRs out of sync unless said PR is changing the file header. Same comment every year ?

avatar wilsonge
wilsonge - comment - 12 Jan 2017

I forgot about bump. My bad!

avatar zero-24
zero-24 - comment - 12 Jan 2017

@jeckodevelopment @wilsonge please add the milestone on merge ;)

avatar wilsonge
wilsonge - comment - 12 Jan 2017

I kinda deliberately didn't - this is just junk in the changelog? I dunno.....

avatar mbabker
mbabker - comment - 12 Jan 2017

Not every pull request needs to go to a milestone. Some changes are administrative in nature and really don't matter. If someone ran a PR updating the version numbers based on running build/bump.php does that really need to be there?

avatar zero-24
zero-24 - comment - 13 Jan 2017

In the past we added every change even just in the tests or build folder to a milestone. If we want to change that just tell me how exactly you want to handle that. So which change need a milestone one which not.

avatar rdeutz
rdeutz - comment - 13 Jan 2017

I am pushing admin changes directly without making a PR, so the junk started as the PR was made. Any PR should have a milestone, test, ....

Add a Comment

Login with GitHub to post a comment