-
Notifications
You must be signed in to change notification settings - Fork 0
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
1. Base project setup #1
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Janos Bonic <[email protected]> Co-authored-by: James Humphries <[email protected]>
faea326
to
1defea4
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.
Internal tools look pretty nice! I left some questions regarding cross-repo standards
Signed-off-by: AbstractionFactory <[email protected]>
c49f140
to
2e4609a
Compare
Signed-off-by: AbstractionFactory <[email protected]>
Signed-off-by: AbstractionFactory <[email protected]>
Signed-off-by: AbstractionFactory <[email protected]>
Signed-off-by: AbstractionFactory <[email protected]>
So the problem this PR is trying to solve code-wise, is to move all of the complexity of running the different tooling of the project to internal tools? I'm trying to think of other solutions that we could use instead of creating a custom go tool for each lightweight wrapper. I also have a preference that when we expose cli args to tooling, we keep the flag names the same. |
@cam72cam it's trying to make it simpler to run the tools we also run in the CI. You can now run An alternative solution could be to create separate folders with only |
This PR adds the base project setup with CI workflows and tools.