-
Notifications
You must be signed in to change notification settings - Fork 4
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
ci: init pipeline #69
Conversation
nice |
tbh it would help if I set up the secrets but it is what it is |
Confirmed that the deploy works - going to close this PR rq to test the delete workflow |
Confirmed delete workflow works 🎉 cc: @andrew-wang0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Epic DevOps!
It looks like I'm running into issues when using the endpoints, but we can stash that as future work
Just investigated and it looks like the bindings aren't being created properly. This should hopefully work. |
@andrew-wang0 would you mind checking a couple of the endpoints again? |
Looks good now 😎 👍 |
Description
Initial version of the CI pipeline, long overdue.
Related Issue
Closes #26.
How Has This Been Tested?
GitHub reads actions workflows frommain
so cannot be tested yet. Should merge this first then iron out any issues that manifest on other PRs when enabled.Just kidding, it works somehow? Let's see if things break.
Types of changes
Checklist: