Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add option to allow certain keys to preserve their order / not be normalised #996

Closed
wants to merge 23 commits into from

Conversation

fredden
Copy link

@fredden fredden commented Nov 9, 2022

This pull request adds the ability for users to specify paths to not normalise / sort.

Related to #985
Fixes #704
Fixes integration test failures in #956

@codecov
Copy link

codecov bot commented Nov 9, 2022

Codecov Report

Merging #996 (95a318a) into main (6efb6bd) will increase coverage by 3.14%.
The diff coverage is 100.00%.

❗ Current head 95a318a differs from pull request most recent head b8ecead. Consider uploading reports for the commit b8ecead to get more accurate results

@@             Coverage Diff              @@
##               main     #996      +/-   ##
============================================
+ Coverage     87.04%   90.18%   +3.14%     
- Complexity       57       71      +14     
============================================
  Files             4        4              
  Lines           193      214      +21     
============================================
+ Hits            168      193      +25     
+ Misses           25       21       -4     
Impacted Files Coverage Δ
src/Command/NormalizeCommand.php 90.81% <100.00%> (+1.86%) ⬆️
src/NormalizePlugin.php 77.77% <0.00%> (+19.44%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@localheinz localheinz self-assigned this Nov 10, 2022
@localheinz
Copy link
Member

Closing in favour of #1056.

@fredden fredden deleted the preserve-order branch February 27, 2023 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make "paths that should not be sorted" configurable
2 participants