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 issue #559, a Windows line endings-related bug #560

Merged
merged 2 commits into from
Dec 7, 2023

Conversation

christian-monch
Copy link
Contributor

@christian-monch christian-monch commented Dec 7, 2023

This commit fixes issue #559.

It allows itemize to any of multiple line-separators to separate the output of git annex find. This takes care of the fact that some git-annex versions will output an unrequested '\r' in addition to a requested '\n'.

This commit fixes issue datalad#559. It allows `itemize`
to any of multiple line-separators to separate the
output of `git annex find`. This takes care of the
fact that some git-annex versions will output
an unrequested `'\r'` in addition to a requested
`'\n'`.
@christian-monch christian-monch requested a review from mih as a code owner December 7, 2023 12:32
Copy link

codecov bot commented Dec 7, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (bd59fce) 92.87% compared to head (872f2b1) 92.88%.
Report is 5 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #560   +/-   ##
=======================================
  Coverage   92.87%   92.88%           
=======================================
  Files         143      143           
  Lines       10365    10366    +1     
  Branches     1141     1141           
=======================================
+ Hits         9627     9628    +1     
  Misses        714      714           
  Partials       24       24           

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

@mih mih merged commit 9cc9171 into datalad:main Dec 7, 2023
9 checks passed
@mih mih added this to the 1.1 milestone Dec 9, 2023
@christian-monch christian-monch deleted the issue-559-key-terminator branch July 16, 2024 10:12
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