Closed jsmentch closed 2 years ago
Fixes Kong2022 readme instructions
I was trying to run the Kong2022 parcellation code and found and fixed some typos in the code of the readme file. No changes to any actual code
For person submit the PR:
[ ] I ran CBIG intermediate unit test just before submitting.
intermediate
[ ] I only ran CBIG light unit test just before submitting, because ( ).
light
[x] I did not run any unit test just before submitting, because ( it is just a readme file ).
[ ] This pull request includes stable project changes. If yes, please answer the following questions.
[ ] I ran the unit test for the changed stable project.
[ ] I did not run the unit test for the changed stable project because ( ).
[ ] I would like to have my unit test included in the intermediate unit test and I have contacted admin.
For person review the PR:
[ ] I ran CBIG intermediate unit test just before merging.
[ ] I only ran CBIG light unit test just before merging, because ( ).
[ ] I did not run any unit test just before merging, because ( ).
@YeoPrivateLab/admins
Thanks for catching those typos! We don't allow making changed in public repo because we update our private repo and update public repo to make a new release. We have integrated the change in the recent release.
Fixes Kong2022 readme instructions
Changes proposed in this pull request:
I was trying to run the Kong2022 parcellation code and found and fixed some typos in the code of the readme file. No changes to any actual code
CBIG Unit Tests (please tick below):
For person submit the PR:
[ ] I ran CBIG
intermediate
unit test just before submitting.[ ] I only ran CBIG
light
unit test just before submitting, because ( ).[x] I did not run any unit test just before submitting, because ( it is just a readme file ).
[ ] This pull request includes stable project changes. If yes, please answer the following questions.
[ ] I ran the unit test for the changed stable project.
[ ] I did not run the unit test for the changed stable project because ( ).
[ ] I would like to have my unit test included in the intermediate unit test and I have contacted admin.
For person review the PR:
[ ] I ran CBIG
intermediate
unit test just before merging.[ ] I only ran CBIG
light
unit test just before merging, because ( ).[ ] I did not run any unit test just before merging, because ( ).
[ ] This pull request includes stable project changes. If yes, please answer the following questions.
[ ] I ran the unit test for the changed stable project.
[ ] I did not run the unit test for the changed stable project because ( ).
@YeoPrivateLab/admins