Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Review DataladDataGrabber #55

Merged
merged 1 commit into from
Sep 13, 2022
Merged

Conversation

bpoldrack
Copy link
Contributor

This patch just adds a few comments to consider and changes the use of datalad install to datalad clone.

@bpoldrack bpoldrack force-pushed the review-datalad-grabber branch from 658ef1f to 052606e Compare September 13, 2022 11:07
Copy link
Contributor

@fraimondo fraimondo left a comment

Choose a reason for hiding this comment

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

Add yourself to the contributors and latest.inc file

@bpoldrack bpoldrack force-pushed the review-datalad-grabber branch from 052606e to 3211556 Compare September 13, 2022 11:21
This patch just adds a few comments to consider and changes the use of
`datalad install` to `datalad clone`.
@bpoldrack bpoldrack force-pushed the review-datalad-grabber branch from 3211556 to f6cd0a4 Compare September 13, 2022 11:31
Copy link
Member

@synchon synchon left a comment

Choose a reason for hiding this comment

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

LGTM

@synchon synchon merged commit ebceb60 into juaml:main Sep 13, 2022
@synchon synchon added the dataset Issues or pull requests related to datagrabbers or datasets label Jul 7, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
dataset Issues or pull requests related to datagrabbers or datasets
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants