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.

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 by Bors] - Use go version specified by go.mod for CI #6560

Closed
wants to merge 1 commit into from

Conversation

fasmat
Copy link
Member

@fasmat fasmat commented Dec 20, 2024

Motivation

The CI still uses a hard coded go version to build and unit test the node code. Both the release and systest jobs already parse and use the version specified in go.mod. This updates the CI to also use the version in go.mod instead of a hard coded setting.

Description

When setting up go on the runner use the version specified in go.mod for build and unit test jobs instead of a hard coded version.

Test Plan

existing tests pass

TODO

  • Explain motivation or link existing issue(s)
  • Test changes and document test plan
  • Update documentation as needed
  • Update changelog as needed

@fasmat fasmat self-assigned this Dec 20, 2024
Copy link

codecov bot commented Dec 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.7%. Comparing base (ef8ba69) to head (21f9916).
Report is 1 commits behind head on develop.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff            @@
##           develop   #6560     +/-   ##
=========================================
- Coverage     79.8%   79.7%   -0.1%     
=========================================
  Files          355     355             
  Lines        47143   47143             
=========================================
- Hits         37635   37618     -17     
- Misses        7371    7386     +15     
- Partials      2137    2139      +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@fasmat
Copy link
Member Author

fasmat commented Dec 20, 2024

bors merge

spacemesh-bors bot pushed a commit that referenced this pull request Dec 20, 2024
## Motivation

The CI still uses a hard coded go version to build and unit test the node code. Both the release and systest jobs already parse and use the version specified in `go.mod`. This updates the CI to also use the version in `go.mod` instead of a hard coded setting.
@spacemesh-bors
Copy link

Build failed:

@fasmat
Copy link
Member Author

fasmat commented Dec 20, 2024

bors merge

spacemesh-bors bot pushed a commit that referenced this pull request Dec 20, 2024
## Motivation

The CI still uses a hard coded go version to build and unit test the node code. Both the release and systest jobs already parse and use the version specified in `go.mod`. This updates the CI to also use the version in `go.mod` instead of a hard coded setting.
@spacemesh-bors
Copy link

Pull request successfully merged into develop.

Build succeeded:

@spacemesh-bors spacemesh-bors bot changed the title Use go version specified by go.mod for CI [Merged by Bors] - Use go version specified by go.mod for CI Dec 20, 2024
@spacemesh-bors spacemesh-bors bot closed this Dec 20, 2024
@spacemesh-bors spacemesh-bors bot deleted the use-go-version-in-go-mod branch December 20, 2024 13:04
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.

3 participants