Skip to content

Conversation

michellewang
Copy link
Collaborator

@michellewang michellewang commented Jul 29, 2025

Changes proposed in this pull request:

  • Use vars instead of env context

Checklist (for reviewers)

This section is for the PR reviewer

  • PR has an interpretable title with a prefix (e.g. [BUG], [DOC], [ENH], [MAINT])
    Refer to NumPy Development Guide for a full list
  • PR links to GitHub issue with mention Closes #XXXX
  • Tests pass
  • Checks pass

For new features:

  • Tests have been added

For bug fixes:

  • There is at least one test that would fail under the original bug conditions

📚 Documentation preview 📚: https://nipoppy--702.org.readthedocs.build/en/702/

@michellewang michellewang requested a review from mathdugre July 29, 2025 16:08
@github-actions github-actions bot added the T.0 - Bug Something isn't working label Jul 29, 2025
@nipoppy-bot nipoppy-bot bot added this to Software Jul 29, 2025
@nipoppy-bot nipoppy-bot bot moved this to Pending review in Software Jul 29, 2025
Copy link

codecov bot commented Jul 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-project-automation github-project-automation bot moved this from Pending review to Approved in Software Jul 29, 2025
@mathdugre mathdugre merged commit 641d4b9 into nipoppy:main Jul 29, 2025
17 checks passed
@nipoppy-bot nipoppy-bot bot moved this from Approved to Done in Software Jul 29, 2025
@michellewang michellewang deleted the 701/bug_api_tests branch August 1, 2025 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T.0 - Bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[BUG] ZENODO_ID not passed to API tests
2 participants