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.

Sign up for GitHub

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

Jump to bottom

Check license data in github action. #42

Merged
merged 2 commits into from
Mar 20, 2025
Merged

Check license data in github action. #42

merged 2 commits into from
Mar 20, 2025

Conversation

flihp
Copy link
Contributor

@flihp flihp commented Mar 18, 2025

No description provided.

@flihp flihp requested review from lzrd and labbott March 18, 2025 22:02
@flihp
Copy link
Contributor Author

flihp commented Mar 18, 2025

Forgot the license header for fwidgen. This is intended to catch such mistakes in PRs.

@labbott
Copy link
Contributor

labbott commented Mar 19, 2025

I think this should be fine, can we triple check in hubris chat to make sure?

@flihp flihp merged commit 1ea9af9 into oxidecomputer:main Mar 20, 2025
4 checks passed
@flihp flihp deleted the license-check branch March 20, 2025 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mkeeter mkeeter mkeeter approved these changes

@lzrd lzrd Awaiting requested review from lzrd

@labbott labbott Awaiting requested review from labbott

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@flihp @labbott @mkeeter