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

fix: clone command in test_trtools.sh testing script #214

Merged
merged 4 commits into from
Feb 21, 2024

Conversation

aryarm
Copy link
Member

@aryarm aryarm commented Feb 19, 2024

This fixes a mistake I made in b23fa56 (in PR #202). I forgot to clone into the current directory and only realized when the bioconda test failed in bioconda/bioconda-recipes#45894 (comment). We managed to patch the bioconda recipe, but anyone who tries to run the test_trtools.sh script after installing TRTools 6.0.0 with pip will still run into an error. This PR fixes that error for the next release of TRTools.

Update (2/20): I also added commit 37c75bb to ensure the test_trtools.sh script can be executed locally in a development installation of TRTools. Of course, this should only be used to test that the test_trtools.sh script itself is working. It won't actually test TRTools properly because it will use an outdated version of our test and data files.

@aryarm aryarm marked this pull request as ready for review February 19, 2024 23:03
@aryarm aryarm changed the title fix: failing test_trtools.sh testing script fix: clone command in test_trtools.sh testing script Feb 20, 2024
Copy link
Contributor

@LiterallyUniqueLogin LiterallyUniqueLogin left a comment

Choose a reason for hiding this comment

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

I committed a small change to one of the comments you wrote. Aside from that, as long as you've checked everything works, this looks good. Feel free to push this if you are okay with my change.

@aryarm aryarm merged commit 756a516 into master Feb 21, 2024
12 checks passed
@aryarm aryarm deleted the fix/testing-script branch February 21, 2024 06:49
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.

2 participants