-
Notifications
You must be signed in to change notification settings - Fork 8
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
Move to deploy-on-release #790
Comments
Let's not setup the auto-release. This repo is going to get its refactor in the not too distant future. So let's just release by hand for now and save ourselves the setup. |
@surchs Did you have something in mind for this? We already have netlify on PRs here like we do in the query tool and the openneuro UI: |
Yes @rmanaem, in fact the nightly builds (whatever the HEAD of main branch is) are already hosted here: https://neurobagel-annotator.netlify.app/ - we just never use that. Maybe we could link this deployment with a nice clear warning that it is unstable/can break from the repo description or README |
Recent events have shown us that deploying on every merge to main is risky, because you can deloy silly things at inopportune times. Let's follow the good examle of our dockerized tools and only deploy to prod on a release.
Specific todos:
[ ] Setup / carry over the auto-release workflow from other projectsThe text was updated successfully, but these errors were encountered: