Skip to content

Libs(Go): get go tests running in CI #1452

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

Merged
merged 1 commit into from
Sep 25, 2024
Merged

Libs(Go): get go tests running in CI #1452

merged 1 commit into from
Sep 25, 2024

Conversation

svix-onelson
Copy link
Contributor

@svix-onelson svix-onelson commented Sep 25, 2024

We've got a handful of unit test that could catch problems when the Go sources are changing, so let's run them as a part of CI.

This doesn't re-run the OpenAPI codegen (but it could later). It would be nice to know if the checked in sources were not aligned with whatever the generator produces, for example if a recent spec bump PR neglected to check them in.

@svix-onelson svix-onelson force-pushed the onelson/go-ci branch 4 times, most recently from f3bc021 to a48de8f Compare September 25, 2024 17:53
@svix-onelson svix-onelson marked this pull request as ready for review September 25, 2024 17:54
@svix-onelson svix-onelson requested a review from a team as a code owner September 25, 2024 17:54
We've got a handful of unit test that could catch problems when the Go
sources are changing, so let's run them as a part of CI.

This doesn't re-run the OpenAPI codegen (but it could later). It would
be nice to know if the checked in sources were not aligned with whatever
the generator produces, for example if a recent spec bump PR neglected to
check them in.
@svix-onelson svix-onelson merged commit d2571f1 into main Sep 25, 2024
3 checks passed
@svix-onelson svix-onelson deleted the onelson/go-ci branch September 25, 2024 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants