-
Notifications
You must be signed in to change notification settings - Fork 17
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
ci: adds basic setup #30
Conversation
be2b3be
to
4d42794
Compare
rebased. I'm not sure if actions are disabled on this repo, but most of the change only helps if they are. |
rebased. main thing left in here is lint fix, fix of tinygo go.mod, and basic CI plumbing |
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.
Thanks!
.github/workflows/commit.yaml
Outdated
steps: | ||
- uses: actions/checkout@v3 | ||
|
||
- uses: actions/setup-go@v3 |
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 like using v4
elsewhere but not here, if it's because of the default cache behavior, maybe can use v4
and cache: false
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.
accident
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.
well you are right ;) missing comment which should be there with the false
This sets up basic CI with a couple notable targets: * `make check`: runs various linters and formatters, and ensures projects build. * `make test-scheduler`: runs scheduler-based unit and e2e tests. Later, we can make tests that run in tinygo for the guest, but as that's tricky I left things here for now. Signed-off-by: Adrian Cole <adrian@tetrate.io> Co-authored-by: Anuraag Agrawal <anuraaga@gmail.com>
thanks @anuraaga I wove in the feedback and preemptively squashed as usually asked to and doing that loses approvals. |
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
approve
Let's see how it works.
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: anuraaga, codefromthecrypt, kerthcet The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
This sets up basic CI with a couple notable targets:
make check
: runs various linters and formatters, and ensures projects build.make test-scheduler
: runs scheduler-based unit and e2e tests.Which issue(s) this PR fixes:
None
Special notes for your reviewer:
Later, we can make tests that run in tinygo for the guest, but as that's tricky I left things here for now.
Does this PR introduce a user-facing change?
NONE
What are the benchmark results of this change?
N/A