Skip to content

Conversation

@thomasferrandiz
Copy link
Collaborator

@thomasferrandiz thomasferrandiz commented Mar 19, 2025

What this PR does / why we need it:
This GHA suffered a security breach last week and is not required.
We can use paths-ignore to avoid running the workflow
when only doc files have been modified.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Special notes for your reviewer (optional):

@coveralls
Copy link

coveralls commented Mar 19, 2025

Pull Request Test Coverage Report for Build 13950346760

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 9 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.2%) to 51.286%

Files with Coverage Reduction New Missed Lines %
pkg/node-controller/controller.go 9 56.67%
Totals Coverage Status
Change from base Build 13947338124: -0.2%
Covered Lines: 1954
Relevant Lines: 3810

💛 - Coveralls

This GHA suffered a security breach last week and is nort required.
We can use paths-ignore to avoid running the workflow
when only doc files have been modified.
@thomasferrandiz thomasferrandiz marked this pull request as ready for review March 19, 2025 15:26
@thomasferrandiz thomasferrandiz changed the title Remove tj actions Remove tj-actions/changed-files Mar 24, 2025
Copy link
Member

@dougbtv dougbtv left a comment

Choose a reason for hiding this comment

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

LGTM! thank you!!

@dougbtv dougbtv merged commit a18f177 into k8snetworkplumbingwg:master Apr 3, 2025
10 of 11 checks passed
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