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: Tolerate missing git #1286

Merged
merged 1 commit into from
Dec 26, 2023
Merged

FIX: Tolerate missing git #1286

merged 1 commit into from
Dec 26, 2023

Conversation

effigies
Copy link
Member

Closes gh-1285.

The alternative would be to catch the exception in the test, but I can't think of a good reason for us to crash the interpreter over this.

Any objections?

Copy link

codecov bot commented Dec 13, 2023

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (f7b9bc4) 92.23% compared to head (46a765d) 92.23%.

Files Patch % Lines
nibabel/pkg_info.py 20.00% 3 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1286   +/-   ##
=======================================
  Coverage   92.23%   92.23%           
=======================================
  Files          99       99           
  Lines       12459    12461    +2     
  Branches     2561     2562    +1     
=======================================
+ Hits        11492    11494    +2     
  Misses        643      643           
  Partials      324      324           

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

@effigies
Copy link
Member Author

I'll consider this reviewed by @yurivict. Thanks!

@effigies effigies merged commit 1436a6e into nipy:master Dec 26, 2023
51 of 52 checks passed
@effigies effigies deleted the fix/missing-git branch December 26, 2023 12:40
effigies added a commit that referenced this pull request Feb 27, 2024
effigies added a commit that referenced this pull request Feb 27, 2024
5.2.1 (Monday 26 February 2024)

Bug-fix release in the 5.2.x series.

Enhancements
------------
* Support "flat" ASCII-encoded GIFTI DataArrays (pr/1298) (PM, reviewed by CM)

Bug fixes
---------
* Tolerate missing ``git`` when reporting version info (pr/1286) (CM, reviewed by
  Yuri Victorovich)
* Handle Siemens XA30 derived DWI DICOMs (pr/1296) (CM, reviewed by YOH and
  Mathias Goncalves)

Maintenance
-----------
* Add tool for generating GitHub-friendly release notes (pr/1284) (CM)
* Accommodate pytest 8 changes (pr/1297) (CM)

* tag '5.2.1':
  REL: 5.2.1
  Backport gh-1296: Conditionally drop derived volumes from DWI sequences
  Backport gh-1298: Support "flat" ASCII-encoded GIFTI DataArrays
  Build(deps): Bump codecov/codecov-action from 3 to 4
  Build(deps): Bump the actions-infrastructure group with 3 updates
  Backport gh-1297: Accommodate pytest 8 changes
  Backport gh-1286: Tolerate missing git
  MNT: Advertise Python 3.12 support
  DOC: Fix intersphinx mapping and reference type
  Backport gh-1284: Add tool for generating GitHub-friendly release notes
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.

1 test fails: test_pkg_info
1 participant