? Success

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
30 May 2016

Summary of Changes

Simple PR. Make bump.php script update the versions and links in readme files also.

Testing Instructions

Code review, or:
1. Apply patch
2. Run php build/bump.php -v 3.7.0in joomla staging root
3. Check the readme files test links are changed

Hint: run php build/bump.php -v 3.6.0-alpha2 to reset to previous version

avatar andrepereiradasilva andrepereiradasilva - open - 30 May 2016
avatar andrepereiradasilva andrepereiradasilva - change - 30 May 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 30 May 2016
Labels Added: ?
avatar andrepereiradasilva andrepereiradasilva - change - 30 May 2016
The description was changed
222adf5 30 May 2016 avatar andrepereiradasilva cs
avatar brianteeman
brianteeman - comment - 30 May 2016

This is not correct. The readme file only refers to 3.x and not 3.x.y. If I read this PR correctly then the full 3.x.y will be used which will cause issues with the links

avatar brianteeman brianteeman - change - 30 May 2016
Category CLI
avatar brianteeman
brianteeman - comment - 30 May 2016

yes I know - that was my PR ;)

My point was that wont this PR change that from 3.x to 3.x.y which will
break the link
`https://docs.joomla.org/Joomla_3.6_version_history

On 30 May 2016 at 18:39, andrepereiradasilva notifications@github.com
wrote:

@brianteeman https://github.com/brianteeman see
https://github.com/joomla/joomla-cms/pull/10193/files


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#10680 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABPH8Z9xYgLC0onxigD3M_dVtfrE_yklks5qGyDWgaJpZM4Ip8Zk
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar andrepereiradasilva
andrepereiradasilva - comment - 30 May 2016

brian it will not change.
please test

avatar andrepereiradasilva
andrepereiradasilva - comment - 30 May 2016
Version data:
- Main:                 3.7
- Release:              3.7.1
- Full:                 3.7.1-alpha1
- Build:
- Dev Level:            1-alpha1
- Dev Status:           Alpha
- Release date:         30-May-2016
- Release time:         18:17
- Release timezone:     GMT
- Creation date:        May 2016

it uses the main version, not the release

avatar brianteeman
brianteeman - comment - 30 May 2016

If I read this PR correctly then the full 3.x.y will be used which will cause issues with the links

And i didnt


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10680.

avatar brianteeman brianteeman - test_item - 30 May 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 30 May 2016

I have tested this item successfully on 222adf5


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10680.

avatar 810 810 - test_item - 31 May 2016 - Tested successfully
avatar 810
810 - comment - 31 May 2016

I have tested this item successfully on 222adf5


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10680.

avatar brianteeman brianteeman - change - 31 May 2016
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 31 May 2016

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10680.

avatar joomla-cms-bot joomla-cms-bot - change - 31 May 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 31 May 2016
Milestone Added:
avatar roland-d roland-d - change - 1 Jun 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-06-01 19:05:09
Closed_By roland-d
avatar roland-d roland-d - close - 1 Jun 2016
avatar roland-d roland-d - merge - 1 Jun 2016
avatar joomla-cms-bot joomla-cms-bot - close - 1 Jun 2016
avatar roland-d roland-d - reference | d38a30d - 1 Jun 16
avatar roland-d roland-d - merge - 1 Jun 2016
avatar roland-d roland-d - close - 1 Jun 2016
avatar joomla-cms-bot joomla-cms-bot - change - 1 Jun 2016
Labels Removed: ?
avatar roland-d
roland-d - comment - 1 Jun 2016

Thanks everybody

Add a Comment

Login with GitHub to post a comment