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

Remove toolchain from go.mod #440

Merged
merged 2 commits into from
Apr 30, 2024
Merged

Conversation

jacobweinstock
Copy link
Member

@jacobweinstock jacobweinstock commented Apr 29, 2024

Description

This causes ci and dependabot issues. Also needed to use the shell.nix file in the CI "Fetch Deps" or newer versions of Go get used and cause changes to the go.mod.

Why is this needed

Fixes: #

How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

Checklist:

I have:

  • updated the documentation and/or roadmap (if required)
  • added unit or e2e tests
  • provided instructions on how to upgrade

This causes ci issues.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
Without this newer Go versions are used
and are causing changes to go.mod

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
Copy link

codecov bot commented Apr 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69%. Comparing base (163fe4e) to head (a5b45cd).

Additional details and impacted files
@@         Coverage Diff         @@
##           main   #440   +/-   ##
===================================
  Coverage    69%    69%           
===================================
  Files        16     16           
  Lines      1386   1386           
===================================
  Hits        958    958           
  Misses      401    401           
  Partials     27     27           

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

@chrisdoherty4
Copy link
Member

Does this change represent a general recommendation?

@jacobweinstock
Copy link
Member Author

Does this change represent a general recommendation?

No. Was just trying to get CI working.

@jacobweinstock jacobweinstock added the ready-to-merge Signal to Mergify to merge the PR. label Apr 30, 2024
@mergify mergify bot merged commit 9839110 into tinkerbell:main Apr 30, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Signal to Mergify to merge the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants