Skip to content

Composer: normalize the file#22121

Merged
jrfnl merged 1 commit intotrunkfrom
JRF/composer-normalize
Mar 21, 2025
Merged

Composer: normalize the file#22121
jrfnl merged 1 commit intotrunkfrom
JRF/composer-normalize

Conversation

@jrfnl
Copy link
Copy Markdown
Contributor

@jrfnl jrfnl commented Mar 21, 2025

Context

  • Tidy up composer.json file

Summary

This PR can be summarized in the following changelog entry:

  • Tidies up composer.json file.

Relevant technical choices:

Well, mostly (scripts are not alphabetized, but still grouped by task).

Note: this is done as a one-time only action. The normalize script will not be run in CI to enforce normalization.

Style has been standardized to --indent-style=tab --indent-size=1.

Ref: https://github.com/ergebnis/composer-normalize

Test instructions

Test instructions for the acceptance test before the PR gets merged

This PR can be acceptance tested by following these steps:

  • N/A

Well, mostly (scripts are not alphabetized, but still grouped by task).

Note: this is done as a one-time only action. The normalize script will **_not_** be run in CI to enforce normalization.

Style has been standardized to `--indent-style=tab --indent-size=1`.

Ref: https://github.com/ergebnis/composer-normalize
@jrfnl jrfnl added yoast cs/qa changelog: non-user-facing Needs to be included in the 'Non-userfacing' category in the changelog labels Mar 21, 2025
@jrfnl jrfnl added this to the 24.9 milestone Mar 21, 2025
@jrfnl jrfnl merged commit 94af3b9 into trunk Mar 21, 2025
29 checks passed
@jrfnl jrfnl deleted the JRF/composer-normalize branch March 21, 2025 13:06
@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 9cedfdafbdfa42f4514b4eb677b77d820abd3620

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 54.896%

Totals Coverage Status
Change from base Build e4596cd641e8a94644d01cb08e8a39144e18a357: 0.0%
Covered Lines: 30835
Relevant Lines: 56623

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog: non-user-facing Needs to be included in the 'Non-userfacing' category in the changelog yoast cs/qa

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants