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 typo #90

Merged
merged 1 commit into from
Dec 2, 2023
Merged

Fix typo #90

merged 1 commit into from
Dec 2, 2023

Conversation

bersbersbers
Copy link
Contributor

Thanks for this feature by the way - I just learned about pytest-order and was about to open a feature request when I saw the 1.2.0 release notes :)

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (337aee1) 97.56% compared to head (c51bc42) 97.56%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #90   +/-   ##
=======================================
  Coverage   97.56%   97.56%           
=======================================
  Files           5        5           
  Lines         575      575           
  Branches      136      136           
=======================================
  Hits          561      561           
  Misses          8        8           
  Partials        6        6           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@mrbean-bremen mrbean-bremen left a comment

Choose a reason for hiding this comment

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

Thanks! I tend to only find typos in other people's writing, not in my own...
Thought your username looked familiar - now found that you've been asking about defer_size in pydicom :)

@mrbean-bremen mrbean-bremen merged commit cf724fa into pytest-dev:main Dec 2, 2023
17 checks passed
@bersbersbers
Copy link
Contributor Author

Thought your username looked familiar

Don't forget https://stackoverflow.com/q/77577740/are-dicom-attributes-always-sorted-by-tag :)

@mrbean-bremen
Copy link
Member

Ah, that was you - I now see that I answered only half the question in my comment...

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.

3 participants