-
Notifications
You must be signed in to change notification settings - Fork 70
Setting up UV #254
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
Setting up UV #254
Conversation
@caleb531 so it appears this was a lot easier than expected. I don't think this is quite in a stage to merge yet (I'm not sure exactly how you want to set up the publish workflow), but I think this is a strong proof of concept so far. I might need your input on how we want to test this fully before merging, since this is a pretty sizeable change, but overall I think simplifies the CI configuration. |
@caleb531 do you have any thoughts about the publish workflow? That's the only one that I haven't yet changed in this PR. The docs building one should work but I haven't tested it. |
@eliotwrobson I think you just need to update the publish workflow to build the project properly with If we want to test this, we can try pushing a dev or alpha prerelease or something. The publish workflow gets triggered when a tag is created, regardless of which branch it is on. |
@caleb531 did you have a chance to take a look at this? If not, I think it's probably ok to just merge and debug later. |
@eliotwrobson Ah, my apologies. It's been a busy few weeks. Let me merge now. |
Resolves #253