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

investigate linter issues #3602

Closed
spencerschrock opened this issue Oct 24, 2023 · 4 comments
Closed

investigate linter issues #3602

spencerschrock opened this issue Oct 24, 2023 · 4 comments

Comments

@spencerschrock
Copy link
Member

spencerschrock commented Oct 24, 2023

Some of these linter warnings aren't firing until after they're merged in. This probably has something to do with the only-new-issue aspect of the linter. Either in our settings or in the config file.

This happened more recently with #3592 and
https://github.com/ossf/scorecard/actions/runs/6620661284/job/17983463822

In the past, this PR (#3558) introduced a linter issue that went unnoticed until it merged to main.

@spencerschrock
Copy link
Member Author

It may have to do with the only-new-issues flag, which doesn't work with push events. I'm curious if it's necessary since our config has:

  new-from-rev: ""

Copy link

This issue is stale because it has been open for 60 days with no activity.

@github-actions github-actions bot added the Stale label Dec 29, 2023
@naveensrinivasan
Copy link
Member

@spencerschrock Can this be closed?

@spencerschrock
Copy link
Member Author

I think it's fine either way, but may as well keep it only-new-issues

if we keep it:
we still aren't blocking any PRs, it's just a matter of the linter failing after push to main, then it can be fixed whenever

if we remove it:
as long as it never breaks, we wont penalize anyone. but if it does, a PR author will need to fix the linter issues before their PR can be merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants