-
Notifications
You must be signed in to change notification settings - Fork 18
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
add misc setup docs and workflows #5
Conversation
NOTE: i expect most of the workflows to fail right now until the initial code is pushed as which point we can start fixing the workflows. |
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.
TY :) small nits on proto generation
removed all protobuf code 👍 |
We are checking lints against Go version 1.20, which might have a conflict with rollkit. |
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.
After rebasing this should be ready for merging.
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.
LGTM
copied over various common workflows and initialization files from rollkit.
with the new auto request workflow, we shouldn't need a CODEOWNERS file as that is covered in the auto request workflow. The auto request won't run until this PR is merged.