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 ci-dev.yaml #162

Merged
merged 2 commits into from
Oct 4, 2023
Merged

Update ci-dev.yaml #162

merged 2 commits into from
Oct 4, 2023

Conversation

ebolyen
Copy link
Collaborator

@ebolyen ebolyen commented Sep 28, 2023

We missed the fact that this has a dependency on q2-types-genomics, so putting it in the shotgun distribution for the moment.

Also, would it be possible to request push access to this repo?

(I need the a manual trigger of this action to run to get this into the 2023.9 release).

@nbokulich
Copy link
Collaborator

can q2-types-genomics also go in core then? Most of the actions in RESCRIPt are for amplicon databases so it would be a little silly to put it in which shotgun (where it has a use, but less) but not in the amplicon/core.

@misialq could you please give @ebolyen push access?

@misialq
Copy link
Contributor

misialq commented Sep 28, 2023

Hey @ebolyen, you should now have access 🚀

@ebolyen
Copy link
Collaborator Author

ebolyen commented Sep 28, 2023

Thanks! Yeah that might be an inevitable conclusion, are you ok with us just figuring out something that will work and then we can shuffle things around for the next cycle to make more sense?

(cc'ing @gregcaporaso as he mentioned we might need to do something like the above)

@nbokulich
Copy link
Collaborator

@ebolyen yeah of course! Makes sense.

@ebolyen
Copy link
Collaborator Author

ebolyen commented Oct 4, 2023

Ok, we're back to green! We'll figure out a plan for moving this back to amplicon for the next cycle.

@ebolyen ebolyen merged commit 9d9d205 into bokulich-lab:master Oct 4, 2023
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.

3 participants