-
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
Miscellaneous fix grab bag #88
Conversation
4a766d7
to
1216039
Compare
bceb730
to
05317a7
Compare
Hooray, finally got actions passing again. This should make future work this pulse go much faster. Everything else I'm going to do is going to be based on this branch so feel free to suggest commits to drop, but ultimately most of them are pretty important but not worth their own PR. |
5e5caa8
to
dbe09f8
Compare
326c39b
to
a08ca67
Compare
a08ca67
to
4e846aa
Compare
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.
A few changes in the comments but feel free to merge once you've addressed those.
Turns out `ubuntu-latest` is also Jammy, whose `protobuf-compiler` version is too low to use the features that are no longer experimental on later releases. So we need the experimental flag.
It's redundant, it's a required dependency of `protobuf-compiler`.
4e846aa
to
9f3cebc
Compare
These were originally all separate PRs, but I ultimately ended up combining them into one because a lot of them require the others (in particular the
dotenvy
one is load bearing).This should fix the CI/CD issue as well.