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

Fix napari-svg version parsing in conftest.py #5947

Merged
merged 3 commits into from
Jun 17, 2023

Conversation

Czaki
Copy link
Collaborator

@Czaki Czaki commented Jun 17, 2023

Description

Fixes problem with parsing napari-svg version as ("0", "1", "10") < ("0", "1", "6") because of using lexicographical compression.

References

Type of change

  • Bug-fix (non-breaking change which fixes an issue)

@Czaki Czaki added the bugfix PR with bugfix label Jun 17, 2023
@Czaki Czaki added this to the 0.4.18 milestone Jun 17, 2023
@Czaki Czaki requested a review from a team June 17, 2023 13:31
@codecov
Copy link

codecov bot commented Jun 17, 2023

Codecov Report

Merging #5947 (f9614d5) into main (c41581c) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #5947   +/-   ##
=======================================
  Coverage   90.19%   90.20%           
=======================================
  Files         615      615           
  Lines       52038    52039    +1     
=======================================
+ Hits        46938    46941    +3     
+ Misses       5100     5098    -2     
Impacted Files Coverage Δ
napari/conftest.py 87.32% <100.00%> (+0.03%) ⬆️

... and 1 file with indirect coverage changes

Copy link
Member

@andy-sweet andy-sweet left a comment

Choose a reason for hiding this comment

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

Much better! Thanks!

@Czaki Czaki changed the title Fix napari-svg vesion parsing in conftest Fix napari-svg version parsing in conftest.py Jun 17, 2023
@Czaki Czaki merged commit 017239d into napari:main Jun 17, 2023
35 checks passed
@Czaki Czaki deleted the fix_napari_svg_version branch June 17, 2023 17:15
Czaki added a commit that referenced this pull request Jun 17, 2023
# Description
Fixes problem with parsing `napari-svg` version as `("0", "1", "10") <
("0", "1", "6")` because of using lexicographical compression.

## Type of change
<!-- Please delete options that are not relevant. -->
- [x] Bug-fix (non-breaking change which fixes an issue)
@Czaki Czaki mentioned this pull request Jun 17, 2023
Czaki added a commit that referenced this pull request Jun 18, 2023
# Description
Fixes problem with parsing `napari-svg` version as `("0", "1", "10") <
("0", "1", "6")` because of using lexicographical compression.

## Type of change
<!-- Please delete options that are not relevant. -->
- [x] Bug-fix (non-breaking change which fixes an issue)
Czaki added a commit that referenced this pull request Jun 19, 2023
# Description
Fixes problem with parsing `napari-svg` version as `("0", "1", "10") <
("0", "1", "6")` because of using lexicographical compression.

## Type of change
<!-- Please delete options that are not relevant. -->
- [x] Bug-fix (non-breaking change which fixes an issue)
Czaki added a commit that referenced this pull request Jun 21, 2023
# Description
Fixes problem with parsing `napari-svg` version as `("0", "1", "10") <
("0", "1", "6")` because of using lexicographical compression.

## Type of change
<!-- Please delete options that are not relevant. -->
- [x] Bug-fix (non-breaking change which fixes an issue)
Czaki added a commit that referenced this pull request Jun 21, 2023
# Description
Fixes problem with parsing `napari-svg` version as `("0", "1", "10") <
("0", "1", "6")` because of using lexicographical compression.

## Type of change
<!-- Please delete options that are not relevant. -->
- [x] Bug-fix (non-breaking change which fixes an issue)
Czaki added a commit that referenced this pull request Jun 21, 2023
# Description
Fixes problem with parsing `napari-svg` version as `("0", "1", "10") <
("0", "1", "6")` because of using lexicographical compression.

## Type of change
<!-- Please delete options that are not relevant. -->
- [x] Bug-fix (non-breaking change which fixes an issue)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix PR with bugfix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants