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 samplesheet location for test profile #25

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

Conversation

adamjtaylor
Copy link
Collaborator

@adamjtaylor adamjtaylor commented Oct 3, 2023

Currently the test profile fails if run from outside the project directory.

This explicitly PR declares in the test config that the test samplesheet is nested under the projectDir located at a public raw github url

@adamjtaylor adamjtaylor self-assigned this Oct 3, 2023
@adamjtaylor
Copy link
Collaborator Author

Moving back to draft while I figure out why this is not working.

@adamjtaylor adamjtaylor marked this pull request as draft October 3, 2023 21:59
@adamjtaylor
Copy link
Collaborator Author

adamjtaylor commented Oct 3, 2023

I had tried to mimic this line from sarek's test.conf

https://github.com/nf-core/sarek/blob/59026dc07633edb83aab3bfb2f65f79db38437a1/conf/test.config#L22C14-L22C27

    input = "${projectDir}/tests/csv/3.0/fastq_single.csv"

@adamjtaylor
Copy link
Collaborator Author

adamjtaylor commented Oct 3, 2023

@adamjtaylor adamjtaylor removed the request for review from thomasyu888 October 3, 2023 22:59
@adamjtaylor
Copy link
Collaborator Author

Decided to point to the raw github url of the samplesheet as this seems more common for nf-core workflows

@adamjtaylor adamjtaylor marked this pull request as ready for review October 3, 2023 23:05
@adamjtaylor
Copy link
Collaborator Author

Looks like we need the test data defined by URL as well

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