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

Update FOSSA action #27

Merged
merged 15 commits into from
Feb 6, 2024
Merged

Update FOSSA action #27

merged 15 commits into from
Feb 6, 2024

Conversation

gab-arrobo
Copy link
Collaborator

No description provided.

thakurajayL
thakurajayL previously approved these changes Feb 2, 2024
@thakurajayL
Copy link
Contributor

Fossa still fails.

@gab-arrobo
Copy link
Collaborator Author

Fossa still fails.

Yes, FOSSA will keep failing until @dafATonf help us configuring the new token as a secret in the omec-project. So, I need to wait for him to help us with this before I open PRs in the other repos replacing/updating the FOSSA action

.github/workflows/main.yml Outdated Show resolved Hide resolved
thakurajayL
thakurajayL previously approved these changes Feb 3, 2024
@dafATonf
Copy link
Contributor

dafATonf commented Feb 3, 2024 via email

.github/workflows/main.yml Outdated Show resolved Hide resolved
@gab-arrobo
Copy link
Collaborator Author

Gabriel -- Moat uawa of github secrets that I've seen use ${{ secrets.FOSSA_OMEC_ORG_PO }}. The variable is surrounded by spaces. I am unable to find an explicit definition that it needs to be done, but worth a try. I'll keep looking to see if it something else comes to mind.
-daf

@dafATonf, the spaces do not seems to be the issue because I added the spaces and I still get the same error

@gab-arrobo gab-arrobo changed the title Update branch name and FOSSA action Update FOSSA action Feb 6, 2024
@gab-arrobo gab-arrobo changed the title Update FOSSA action Update FOSSA action Feb 6, 2024
@gab-arrobo gab-arrobo merged commit f988e55 into omec-project:main Feb 6, 2024
6 checks passed
@gab-arrobo gab-arrobo deleted the fix-gha branch February 6, 2024 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants