PR-4.3-dev Pending

User tests: Successful: Unsuccessful:

avatar obuisard
obuisard
5 Feb 2023

DEPLOY_VERSION is not replaced because of typo.

Summary of Changes

Added a missing _

Testing Instructions

Build a new version and check the comment @SInCE of $linked.

Actual result BEFORE applying this Pull Request

DEPLOY_VERSION is not replaced once the version is built.

Expected result AFTER applying this Pull Request

DEPLOY_VERSION is replaced with the current version number.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar joomla-cms-bot joomla-cms-bot - change - 5 Feb 2023
Category Libraries
avatar obuisard obuisard - open - 5 Feb 2023
avatar obuisard obuisard - change - 5 Feb 2023
Status New Pending
avatar obuisard obuisard - change - 5 Feb 2023
The description was changed
avatar obuisard obuisard - edited - 5 Feb 2023
avatar obuisard obuisard - change - 5 Feb 2023
The description was changed
avatar obuisard obuisard - edited - 5 Feb 2023
avatar laoneo
laoneo - comment - 5 Feb 2023

Should we not hardcode it with the correct value?

avatar laoneo laoneo - close - 6 Feb 2023
avatar laoneo laoneo - merge - 6 Feb 2023
avatar laoneo laoneo - change - 6 Feb 2023
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-02-06 07:47:05
Closed_By laoneo
Labels Added: PR-4.3-dev
avatar laoneo
laoneo - comment - 6 Feb 2023

Thanks!

avatar obuisard
obuisard - comment - 6 Feb 2023

Thank you Allon @laoneo, but you merged this on a branch that is in a freeze state.

avatar heelc29
heelc29 - comment - 6 Feb 2023

@obuisard @laoneo This should be fixed in 4.2 branch; introduced with #39632 in 4.2.7

avatar laoneo
laoneo - comment - 6 Feb 2023

Thank you Allon @laoneo, but you merged this on a branch that is in a freeze state.

I realized it afterwards and informed on mattermost already.

avatar laoneo
laoneo - comment - 6 Feb 2023

@obuisard @laoneo This should be fixed in 4.2 branch; introduced with #39632 in 4.2.7

You are right.

Add a Comment

Login with GitHub to post a comment