8000 ci: fossa workflow by mdtro · Pull Request #922 · getsentry/seer · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

ci: fossa workflow #922

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
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

ci: fossa workflow #922

wants to merge 2 commits into from

Conversation

mdtro
Copy link
Member
@mdtro mdtro commented Jul 17, 2024 •
edited
Loading

Setup FOSSA CI to ensure we are compliant with our open source licensing.

@mdtro mdtro force-pushed the mdtro/fossa branch 2 times, most recently from 4851172 to a87ddab Compare July 24, 2024 20:29
@mdtro mdtro requested review from trillville and jennmueng July 26, 2024 21:02
jennmueng
jennmueng approved these changes Jul 26, 2024
View reviewed changes
Copy link
Member
@jennmueng jennmueng left a comment

Choose a reason for hiding this comment

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

LGTM, but looks like we should get a license setup with johen before we merge this

@mdtro mdtro requested a review from a team as a code owner April 30, 2025 18:39
github-advanced-security[bot]
github-advanced-security bot found potential problems Apr 30, 2025
View reviewed changes
.github/workflows/enforce-license-compliance.yml
runs-on: ubuntu-latest
steps:
- name: 'Enforce License Compliance'
uses: getsentry/action-enforce-license-compliance@main

Check warning

Code scanning / CodeQL

Unpinned tag for a non-immutable Action in workflow Medium

Unpinned 3rd party Action 'Enforce License Compliance' step
Uses Step
Loading
uses 'getsentry/action-enforce-license-compliance' with ref 'main', not a pinned commit hash
@seer-by-sentry Seer by Sentry
Copy link
Contributor

✅ Sentry found no issues in your recent changes ✅

@mdtro
Copy link
Member Author
mdtro commented Apr 30, 2025

@jennmueng -- Did we ever get a license on johen and a new release published so that we can merge this?

@jennmueng
Copy link
Member

@jennmueng -- Did we ever get a license on johen and a new release published so that we can merge this?

I will bother @corps soon, right now hes out fishing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jennmueng jennmueng jennmueng approved these changes

@trillville trillville Awaiting requested review from trillville

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

Successfully merging this pull request may close these issues.
2 participants
@mdtro @jennmueng
0