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

Maintain stable sort order for lists which include sigils #57

Merged
merged 2 commits into from
Dec 18, 2021

Conversation

oalders
Copy link
Member

@oalders oalders commented Dec 17, 2021

  • Ignore sigils when sorting imported symbols
  • Maintain stable sort order for lists which include sigils

@oalders oalders force-pushed the sort-imports branch 2 times, most recently from a5fa9c1 to 0eb6205 Compare December 17, 2021 23:24
@codecov-commenter
Copy link

codecov-commenter commented Dec 17, 2021

Codecov Report

Merging #57 (83529ea) into main (a0afdd8) will increase coverage by 0.26%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #57      +/-   ##
==========================================
+ Coverage   82.78%   83.05%   +0.26%     
==========================================
  Files           8        8              
  Lines         703      714      +11     
  Branches      158      159       +1     
==========================================
+ Hits          582      593      +11     
  Misses         83       83              
  Partials       38       38              
Impacted Files Coverage Δ
lib/App/perlimports/Include.pm 84.26% <100.00%> (+0.93%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a0afdd8...83529ea. Read the comment docs.

@oalders oalders merged commit 0bee73a into main Dec 18, 2021
@oalders oalders deleted the sort-imports branch December 18, 2021 02:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants