Skip to content

Conversation

@ivanvc
Copy link
Member

@ivanvc ivanvc commented Mar 6, 2025

Replace go list -m with go mod edit -json, as the latter can return the same information. This will be helpful when the project migrates to using a Go workspace, as it will return the current module defined in go.mod rather than all the modules from the current directory (using a workspace, the top-level go.mod will return all the child modules from the repository).

Spun off from #19423.
Part of #18409.

Please read https://github.com/etcd-io/etcd/blob/main/CONTRIBUTING.md#contribution-flow.

Replace `go list -m` with `go mod edit -json`, as the latter can return
the same information. This will be helpful when the project migrates to
using a Go workspace, as it will return the current module defined in
go.mod rather than all the modules from the current directory (using a
workspace, the top-level go.mod will return all the child modules from
the repository).

Signed-off-by: Ivan Valdes <[email protected]>
@ivanvc ivanvc force-pushed the use-go-mod-instead-of-go-list-in-release-scripts branch from 50b3f18 to 7e66d3a Compare March 6, 2025 06:50
@codecov
Copy link

codecov bot commented Mar 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.80%. Comparing base (930bf4c) to head (7e66d3a).
Report is 6 commits behind head on main.

Additional details and impacted files

see 16 files with indirect coverage changes

@@            Coverage Diff             @@
##             main   #19546      +/-   ##
==========================================
- Coverage   68.83%   68.80%   -0.04%     
==========================================
  Files         421      421              
  Lines       35901    35901              
==========================================
- Hits        24713    24702      -11     
- Misses       9768     9776       +8     
- Partials     1420     1423       +3     

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 930bf4c...7e66d3a. Read the comment docs.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@k8s-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ahrtr, ivanvc, serathius

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@serathius serathius merged commit 9f1709e into etcd-io:main Mar 6, 2025
32 checks passed
@ivanvc ivanvc deleted the use-go-mod-instead-of-go-list-in-release-scripts branch March 6, 2025 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants