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: make get_snakefile return rel path to snakefile #40

Merged
merged 1 commit into from Apr 6, 2024

Conversation

cademirch
Copy link
Contributor

Should fix #39.

@cademirch cademirch changed the title make get_snakefile return rel path to snakefile fix: make get_snakefile return rel path to snakefile Apr 6, 2024
@vsoch
Copy link
Collaborator

vsoch commented Apr 6, 2024

@cademirch this isn't your fault:

By default, secrets are not passed to workflows triggered from forks, including Dependabot.

Let me make a fork for you to PR to, we will merge there and then PR with that fork. We probably need to figure out a means for this to work in the future, but right now that PR will be the point where we can evaluate the code isn't being malicious, etc.

@vsoch
Copy link
Collaborator

vsoch commented Apr 6, 2024

@vsoch vsoch changed the base branch from main to fix-snakefile-path April 6, 2024 01:45
@vsoch
Copy link
Collaborator

vsoch commented Apr 6, 2024

oh looks like I can do it!

@cademirch
Copy link
Contributor Author

Yeah I figured it wasn't me :) thanks!

@vsoch vsoch merged commit 13c231b into snakemake:fix-snakefile-path Apr 6, 2024
4 of 6 checks passed
vsoch added a commit that referenced this pull request Apr 6, 2024
Should fix #39. Previously #40

Co-authored-by: Cade Mirchandani <cmirchan@ucsc.edu>
vsoch pushed a commit that referenced this pull request Apr 6, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.3.3](v0.3.2...v0.3.3)
(2024-04-06)


### Bug Fixes

* make get_snakefile return rel path to snakefile
([#40](#40))
([#41](#41))
([de2b1da](de2b1da))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

Issues with include
2 participants