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

Clean up shell.nix #726

Merged
merged 2 commits into from
May 10, 2023
Merged

Clean up shell.nix #726

merged 2 commits into from
May 10, 2023

Conversation

jacobweinstock
Copy link
Member

Description

clean up shell.nix file of unused/needed deps for CI.

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

@codecov
Copy link

codecov bot commented May 9, 2023

Codecov Report

Merging #726 (0864ad5) into main (1150a94) will not change coverage.
The diff coverage is n/a.

❗ Current head 0864ad5 differs from pull request most recent head c28e3af. Consider uploading reports for the commit c28e3af to get more accurate results

@@           Coverage Diff           @@
##             main     #726   +/-   ##
=======================================
  Coverage   55.54%   55.54%           
=======================================
  Files          26       26           
  Lines        1100     1100           
=======================================
  Hits          611      611           
  Misses        474      474           
  Partials       15       15           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@@ -9,29 +9,11 @@ in { pkgs ? import (_pkgs.fetchFromGitHub {

with pkgs;

let
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How does this work with the CI now?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dont know if I understand the questions, sorry. Currently, the shell.nix file is only needed for the ci-checks.sh script.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That answers the question 👍🏻 ; I'd forgotten.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
@chrisdoherty4 chrisdoherty4 added the ready-to-merge Signal to Mergify to merge the PR. label May 10, 2023
@chrisdoherty4
Copy link
Member

@Mergifyio queue

@mergify
Copy link
Contributor

mergify bot commented May 10, 2023

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 7be72c7

@mergify mergify bot merged commit 7be72c7 into tinkerbell:main May 10, 2023
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