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

Run codespell on some obvious typos #563

Merged
merged 1 commit into from
May 10, 2022
Merged

Conversation

yarikoptic
Copy link
Member

Primarily goal of this PR is to kick out a 0.11.0 release since we are already at v0.10.0-146-g0da0323 with a good number of merged PRs. I was hoping to finish #544 but haven't yet, so another MINOR major ;) might follow not too far away. Not sure if git foo is correct but at large here are the following pull request merges (kick out low #ed since those are PRs into PRs)

(git)lena:~/proj/heudiconv/heudiconv-master[enh-release]git
$> git log --pretty=oneline --merges v0.10.0..origin/master | grep 'pull request' | nl
     1	0da03238898426538b54160a731809f0de6cd04b Merge pull request #558 from nipy/gh-557
     2	a1f63de34f23a3f6a2a750caaeb9631191988978 Merge pull request #547 from cbinyu/fix_echo_order_in_filename
     3	772feeba1fe87d0bc84d85a9a07ce20344ef46c2 Merge pull request #555 from dbic/rf-3.7
     4	80713f8e2a4e0f25e40eaed28cc5a9ed0c3b5ac9 Merge pull request #553 from dbic/bf-misc
     5	cb2fd912b8310d7022e9358a84c3b589fe3c07e8 Merge pull request #546 from dbic/adds_populate_intended_for
     6	e747f0595dd393384d65219f5137d42b2a5c8e73 Merge pull request #482 from cbinyu/adds_populate_intended_for
     7	92eecfe237069b1d8a6e487067c86c0d37bc0664 Merge pull request #461 from tsalo/fix/updater-lists
     8	4b23544c96c33d05f34f3c839ac12fb51974dbdc Merge pull request #13 from neurorepro/adds_populate_intended_for
     9	717f500bf4a2c2a182a9e92d02aec1a57e8dc1c8 Merge pull request #12 from neurorepro/adds_populate_intended_for
    10	3f9a504270f83d00bb483c8f52ae0c228fc7d808 Merge pull request #535 from UNFmontreal/fix/tar_unordered_files
    11	eddb35ca375978479b621d3ad34d15c93b326ef0 Merge pull request #542 from cbinyu/fix_echo_order_in_filename
    12	80a65385ef867124611d037fceac27d27e30f480 Merge pull request #534 from dbic/bf-warning-test
    13	8d85f14261025fcd7ac2382721ab008f276b715c Merge pull request #533 from dbic/rf-newer-travis
    14	2695ecf5a120fe2b84203e7ab66c67829650641f Merge pull request #529 from dbic/enh-version
    15	c4956b35b38d506a181e0f933a22819b8e687991 Merge pull request #511 from dbic/bf-anon-cmd
    16	0ecd867bb07d29a1412c15d027111f377b33bc00 Merge pull request #10 from dbic/adds_populate_intended_for
    17	141f6e9d78fe591eb2c2f74ea7f151f24b43a73d Merge pull request #34 from cbinyu/patch_dbic
    18	d3b82101c9f00ebc55b2cc0c541ced491aaae953 Merge pull request #11 from cbinyu/BIDSFile_helper
    19	7edcf3627502a0fed89bdf892f08a5e435ad63a7 Merge pull request #4 from nipy/master

so -- if CI is still happy -- will merge and will see if auto and docker building established by @jwodder would work out cleanly.

=== Do not change lines below ===
{
 "chain": [],
 "cmd": "codespell -w heudiconv/queue.py heudiconv/parser.py heudiconv/convert.py heudiconv/tests/test_heuristics.py heudiconv/cli/monitor.py heudiconv/heuristics/reproin.py",
 "exit": 0,
 "extra_inputs": [],
 "inputs": [],
 "outputs": [],
 "pwd": "."
}
^^^ Do not change lines above ^^^
@yarikoptic yarikoptic added release Create a release when this pr is merged internal Changes only affect the internal API labels May 10, 2022
@yarikoptic
Copy link
Member Author

attn @pvelasco @tsalo and @andrew-yian-sun in ref to #541 -- release is imminent ;)

@codecov
Copy link

codecov bot commented May 10, 2022

Codecov Report

Merging #563 (8775507) into master (0da0323) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #563   +/-   ##
=======================================
  Coverage   81.26%   81.26%           
=======================================
  Files          41       41           
  Lines        3817     3817           
=======================================
  Hits         3102     3102           
  Misses        715      715           
Impacted Files Coverage Δ
heudiconv/cli/monitor.py 31.52% <ø> (ø)
heudiconv/convert.py 87.22% <ø> (ø)
heudiconv/heuristics/reproin.py 83.92% <ø> (ø)
heudiconv/parser.py 92.70% <ø> (ø)
heudiconv/tests/test_heuristics.py 100.00% <ø> (ø)
heudiconv/queue.py 90.00% <100.00%> (ø)

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 0da0323...8775507. Read the comment docs.

@yarikoptic yarikoptic merged commit 7de58c1 into nipy:master May 10, 2022
@github-actions
Copy link

🚀 PR was released in v0.11.0 🚀

@yarikoptic yarikoptic deleted the enh-release branch June 6, 2022 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal Changes only affect the internal API release Create a release when this pr is merged released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant