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

ci: add go mod tidy checker #963

Merged
merged 5 commits into from
Jul 12, 2023
Merged

ci: add go mod tidy checker #963

merged 5 commits into from
Jul 12, 2023

Conversation

thehowl
Copy link
Member

@thehowl thehowl commented Jul 10, 2023

#937 added some extra dependencies not needed, probably because of temporary changes for graphing.

I ran go.mod and added a CI check. I've temporarily re-introduced an extra dependency to go.mod to see if the CI check fails correctly.

Checklists...

Contributors Checklist

  • Added new tests, or not needed, or not feasible
  • Provided an example (e.g. screenshot) to aid review or the PR is self-explanatory
  • Updated the official documentation or not needed
  • No breaking changes were made, or a BREAKING CHANGE: xxx message was included in the description
  • Added references to related issues and PRs
  • Provided any useful hints for running manual tests
  • Added new benchmarks to generated graphs, if any. More info here.

Maintainers Checklist

  • Checked that the author followed the guidelines in CONTRIBUTING.md
  • Checked the conventional-commit (especially PR title and verb, presence of BREAKING CHANGE: in the body)
  • Ensured that this PR is not a significant change or confirmed that the review/consideration process was appropriate for the change

@thehowl thehowl requested a review from a team as a code owner July 10, 2023 10:32
@moul moul merged commit 02b88c0 into gnolang:master Jul 12, 2023
73 checks passed
Doozers pushed a commit to Doozers/gno that referenced this pull request Aug 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🌟 Wanted for Launch
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants