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

Add SPDX header #841

Merged
merged 4 commits into from
May 13, 2024
Merged

Add SPDX header #841

merged 4 commits into from
May 13, 2024

Conversation

AndyScherzinger
Copy link
Member

and GH action to check PRs

@blizzz
Copy link
Member

blizzz commented May 10, 2024

Seems there is something wrong with the dependabot file.

@AndyScherzinger
Copy link
Member Author

Seems there is something wrong with the dependabot file.

@blizzz actually that issue is present even before this PR and the changes in the file have no effect on this. If you revert any change in the dependabot file you will still see the exact same error. See dependabot/dependabot-core#8948 for others facing the same issue...

@AndyScherzinger
Copy link
Member Author

Add 7f33a3e to show it breaks anyways @blizzz

@blizzz
Copy link
Member

blizzz commented May 10, 2024

Seems to be a layer 8 origin. #751 (comment)

It did not turn red before though.

Signed-off-by: Andy Scherzinger <[email protected]>
Signed-off-by: Andy Scherzinger <[email protected]>
Signed-off-by: Andy Scherzinger <[email protected]>
@blizzz blizzz force-pushed the chore/noid/spdx branch 2 times, most recently from 79d83a9 to 7ff1204 Compare May 10, 2024 14:50
@blizzz
Copy link
Member

blizzz commented May 10, 2024

Add 7f33a3e to show it breaks anyways @blizzz

dropped the commit again

@AndyScherzinger
Copy link
Member Author

Thanks @blizzz

All green now 💚

@blizzz blizzz merged commit f8c93e1 into master May 13, 2024
50 checks passed
@blizzz blizzz deleted the chore/noid/spdx branch May 13, 2024 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants