Skip to content

Comments

Package version bump#41678

Merged
MacJoom merged 6 commits intojoomla:4.4-devfrom
Digital-Peak:package/version
Sep 11, 2023
Merged

Package version bump#41678
MacJoom merged 6 commits intojoomla:4.4-devfrom
Digital-Peak:package/version

Conversation

@laoneo
Copy link
Member

@laoneo laoneo commented Sep 8, 2023

Pull Request for pr #40513.

Summary of Changes

Updates the version in the package.json file when the Joomla version is bumped. Additionally it uses 4 spaces for indentation, similar to the composer.json file.

Testing Instructions

On the command line in the Joomla root folder execute the following command:
php build/bump.php -v 5.0.0

Actual result BEFORE applying this Pull Request

The version tag in the package.json file contains the old version 4.0.0.

Expected result AFTER applying this Pull Request

The version tag in the package.json file contains the old version 5.0.0.

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

@laoneo laoneo marked this pull request as draft September 8, 2023 12:29
Copy link
Contributor

@wilsonge wilsonge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine on review

@joomla-cms-bot joomla-cms-bot added the NPM Resource Changed This Pull Request can't be tested by Patchtester label Sep 11, 2023
@laoneo laoneo marked this pull request as ready for review September 11, 2023 06:36
@MacJoom MacJoom merged commit 8a036ef into joomla:4.4-dev Sep 11, 2023
@MacJoom MacJoom added this to the Joomla! 4.4.0 milestone Sep 11, 2023
@MacJoom MacJoom deleted the package/version branch September 11, 2023 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.4-dev

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants