-
Notifications
You must be signed in to change notification settings - Fork 7
overhaul CI with Makefile #52
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
Conversation
f702d6c to
d91040a
Compare
|
@joncinque this felt like the more exhausting but long-term better way to go about getting a publish script working to release the new interface crate. Lmk! |
joncinque
left a comment
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.
Just a couple of comments, looks good otherwise!
joncinque
left a comment
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.
Just a couple of last things
joncinque
left a comment
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.
Looks great!
|
Since this PR renamed some of the CI steps, I had to disable them as required to merge. Once it lands, I'll update the required steps to include the new names. |
Do you have to always do this manually? That's kind of annoying. I was thinking they would fix themselves after this lands. |
Takes the exact same changes as solana-program/system#51 and applies them to the Config program repo.