Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

chore(ci): remove FOSSA workflow due to expired API token #250

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
Merged
SoulPancake merged 1 commit into from
Nov 6, 2025
Merged

chore(ci): remove FOSSA workflow due to expired API token #250

SoulPancake merged 1 commit into from
Nov 6, 2025

Conversation

@SoulPancake
Copy link
Member

@SoulPancake SoulPancake commented Nov 5, 2025 •
edited
Loading

Description

Removing the FOSSA workflow which is always failing due to expired API token.

What problem is being solved?

#243

How is it being solved?

What changes are made to solve it?

References

Review Checklist

  • I have clicked on "allow edits by maintainers".
  • I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
  • The correct base branch is being used, if not main
  • I have added tests to validate that the change in functionality is working as expected

Summary by CodeRabbit

  • Chores
    • Removed CI/CD workflow infrastructure used for compliance scanning and testing.

Copilot AI review requested due to automatic review settings November 5, 2025 05:32
@SoulPancake SoulPancake requested a review from a team as a code owner November 5, 2025 05:32
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 5, 2025 •
edited
Loading

Walkthrough

Deleted the GitHub Actions workflow file .github/workflows/fossa.yaml that performed FOSSA security scans and tests. The workflow was triggered on push to main and manual dispatch, executing repository checkout, JDK setup, Gradle build, and FOSSA scan uploads.

Changes

Cohort / File(s) Change Summary
FOSSA workflow removal
\.\github/workflows/fossa\.yaml		 Deleted entire FOSSA CI workflow configuration including scan uploads and test execution

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • rhamzeh

Pre-merge checks and finishing touches

✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: removing the FOSSA workflow due to an expired API token, which aligns with the changeset that deletes .github/workflows/fossa.yaml.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/remove-fossa-workflow

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI reviewed Nov 5, 2025
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes the FOSSA workflow entirely from the repository. FOSSA is a dependency scanning and license compliance tool, and this change eliminates automated license scanning and compliance checks from the CI/CD pipeline.

Key Changes:

  • Deletion of the entire FOSSA GitHub Actions workflow file

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov-commenter
Copy link

codecov-commenter commented Nov 5, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 36.39%. Comparing base (b69eb86) to head (1cd19fe).

Additional details and impacted files 
@@            Coverage Diff            @@
##               main     #250   +/-   ##
=========================================
  Coverage     36.39%   36.39%           
  Complexity     1141     1141           
=========================================
  Files           188      188           
  Lines          7185     7185           
  Branches        822      822           
=========================================
  Hits           2615     2615           
  Misses         4465     4465           
  Partials        105      105

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@SoulPancake SoulPancake linked an issue Nov 5, 2025 that may be closed by this pull request
FOSSA check in workflow failing due to expired API token #243
Closed
Siddhant-K-code
Siddhant-K-code reviewed Nov 5, 2025
View reviewed changes
Copy link
Member

@Siddhant-K-code Siddhant-K-code left a comment

Choose a reason for hiding this comment

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

I am unsure we should remove this, because it is just a matter of refreshing the FOSSA API Token which @aaguiarz / @rhamzeh can do that!

@SoulPancake SoulPancake added this pull request to the merge queue Nov 6, 2025
Merged via the queue into main with commit d6d5908 Nov 6, 2025
37 checks passed
@SoulPancake SoulPancake deleted the chore/remove-fossa-workflow branch November 6, 2025 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Siddhant-K-code Siddhant-K-code Siddhant-K-code left review comments

Copilot code review Copilot Copilot left review comments

@ttrzeng ttrzeng ttrzeng approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

FOSSA check in workflow failing due to expired API token

5 participants

@SoulPancake @codecov-commenter @ttrzeng @Siddhant-K-code
Morty Proxy This is a proxified and sanitized view of the page, visit original site.