Skip to content
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.

Sign up for GitHub

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

Jump to bottom

chore(test vectors): Add interop test in Go with CI #731

Merged
merged 16 commits into from
Jan 3, 2025
Merged

chore(test vectors): Add interop test in Go with CI #731

merged 16 commits into from
Jan 3, 2025
+376 −15

Conversation

rishav-karanjit
Copy link
Member

@rishav-karanjit rishav-karanjit commented Dec 30, 2024 •
edited
Loading

Issue #, if available:

Description of changes:

Same as #706 but:

  • Its into the mainline branch
  • This PR adds CI for test vectors
  • I have added new make file target name purge_polymorph_code because CI also needs to remove CompanionStruct_Default___ from shim. Now _polymorph_go runs purge_polymorph_code

Squash/merge commit message, if applicable:

chore(test vectors): Add interop test in Go with CI

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@rishav-karanjit rishav-karanjit changed the title chore(testvectors): Add interop test in Go with CI chore(test vectors): Add interop test in Go with CI Dec 30, 2024
@rishav-karanjit rishav-karanjit changed the title chore(test vectors): Add interop test in Go with CI chore(test vectors): Add interop test in Go with CI Dec 30, 2024
@rishav-karanjit rishav-karanjit marked this pull request as ready for review January 3, 2025 17:47
@rishav-karanjit rishav-karanjit requested a review from a team as a code owner January 3, 2025 17:47
@rishav-karanjit rishav-karanjit merged commit d04ddc0 into mainline Jan 3, 2025
83 checks passed
@rishav-karanjit rishav-karanjit deleted the golang-testvector branch January 3, 2025 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lucasmcdonald3 lucasmcdonald3 lucasmcdonald3 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rishav-karanjit @lucasmcdonald3