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

Update test_cli.py #431

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

Update test_cli.py #431

wants to merge 12 commits into from

Conversation

tsalo
Copy link
Member

@tsalo tsalo commented Feb 11, 2025

Closes none. I tried using Cursor to write a bunch of tests for the command-line interfaces, but they didn't work very well.

Changes proposed in this pull request

Documentation that should be reviewed

@tsalo
Copy link
Member Author

tsalo commented Feb 13, 2025

I cannot figure out why CodeCov won't recognize the report. I've checked the XML file and compared it to one from XCP-D, and it looks fine. The only difference I could see was the path to the package on the CircleCI machine (/home/circleci/src/CuBIDS for CuBIDS and /src/xcp_d for XCP-D), but I don't see why that would be a problem.

@tsalo tsalo marked this pull request as ready for review February 13, 2025 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant