Skip to content

Add GitHub test action#85

Merged
sreichl merged 7 commits intomainfrom
gh-test-action
Apr 1, 2026
Merged

Add GitHub test action#85
sreichl merged 7 commits intomainfrom
gh-test-action

Conversation

@hoferand
Copy link
Copy Markdown
Collaborator

@hoferand hoferand commented Apr 1, 2026

@hoferand hoferand marked this pull request as ready for review April 1, 2026 08:09
@hoferand hoferand requested a review from sreichl April 1, 2026 08:09
@sreichl
Copy link
Copy Markdown
Collaborator

sreichl commented Apr 1, 2026

this is really cool and simple! thanks for looking into that and setting it up. interesting that no containerization is required and it still works! FYI the action run cost €0.08 =)

@sreichl sreichl merged commit da1bbce into main Apr 1, 2026
@sreichl sreichl deleted the gh-test-action branch April 1, 2026 12:08
@hoferand
Copy link
Copy Markdown
Collaborator Author

hoferand commented Apr 2, 2026

interesting that no containerization is required and it still works!

Yeah, it’s really nice that the Snakemake action takes care of all of that.

FYI the action run cost €0.08 =)

Nice to know! So, running the action on every push (instead of triggering it manually) is probably cheaper in total. 😄

@sreichl
Copy link
Copy Markdown
Collaborator

sreichl commented Apr 2, 2026

Nice to know! So, running the action on every push (instead of triggering it manually) is probably cheaper in total. 😄

Unsure. Let's keep it manual for now ( I'm also a bit annoyed by getting an email whenever some test fails and I'm not interested in fixing it rn. For example, linting. ^^).

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