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.

Sign up for GitHub

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

Jump to bottom

ENH: Add PyBIDS leading dot warning #211

Merged
merged 2 commits into from
Jun 2, 2022
Merged

ENH: Add PyBIDS leading dot warning #211

merged 2 commits into from
Jun 2, 2022

Conversation

jhlegarreta
Copy link
Contributor

@jhlegarreta jhlegarreta commented Jun 1, 2022 •
edited
Loading

  • ENH: Remove unused import statement
  • ENH: Bump PyBIDS version and add leading dot to extensions

Fixes #171.

@jhlegarreta
ENH: Remove unused import statement
78141b0 
Remove unused import statement.
@jhlegarreta
ENH: Bump PyBIDS version and add leading dot to extensions
b6249eb 
Bump PyBIDS version and add leading dot to extensions.
@jhlegarreta jhlegarreta requested review from kaitj and josephmje June 1, 2022 15:01
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@jhlegarreta jhlegarreta added the type:enhancement Propose enhancement to the lesson label Jun 1, 2022
@netlify
Copy link

netlify bot commented Jun 1, 2022

✅ Deploy Preview for carpentries-dmri ready!

Name Link
🔨 Latest commit b6249eb
🔍 Latest deploy log https://app.netlify.com/sites/carpentries-dmri/deploys/62977f4e1f79400009231cc6
😎 Deploy Preview https://deploy-preview-211--carpentries-dmri.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@jhlegarreta jhlegarreta merged commit 567bc8f into carpentries-incubator:main Jun 2, 2022
@jhlegarreta jhlegarreta deleted the AddPyBIDSLeadingDotWarning branch June 2, 2022 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kaitj kaitj kaitj approved these changes

@josephmje josephmje Awaiting requested review from josephmje

Assignees
No one assigned
Labels
type:enhancement Propose enhancement to the lesson
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

pybids leading dot warning
2 participants
@jhlegarreta @kaitj