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

Deploy FOSSA license scanning #808

Closed
caniszczyk opened this issue May 7, 2020 · 12 comments · Fixed by #860 or #937
Closed

Deploy FOSSA license scanning #808

caniszczyk opened this issue May 7, 2020 · 12 comments · Fixed by #860 or #937
Labels
feature-request All issues for new features that have not been committed to needs-discussion

Comments

@caniszczyk
Copy link

We should enable license scanning on this repo:
https://github.com/fossas/fossa-cli

Ihor will take care of this

@caniszczyk caniszczyk added needs-discussion feature-request All issues for new features that have not been committed to labels May 7, 2020
@idvoretskyi
Copy link
Contributor

cc

@idvoretskyi
Copy link
Contributor

Added via #860

@zroubalik
Copy link
Member

The FOSSA check doesn't work on PRs: #765 (comment)

@idvoretskyi
Copy link
Contributor

@zroubalik I'll check it out, thanks for the heads up.

@idvoretskyi
Copy link
Contributor

@zroubalik an update on this - I'm already in touch with the FOSSA support team on finding the best possible solution. As of now, the most valid solution would be using the push-only token embedded into the workflow file directly (https://docs.fossa.com/docs/api-reference#push-only-api-token), but I'd like to discover if there are any alternative ways.

@idvoretskyi idvoretskyi mentioned this issue Jun 11, 2020
3 tasks
@zroubalik
Copy link
Member

@idvoretskyi thanks for the update!

@idvoretskyi
Copy link
Contributor

We've scheduled a call with FOSSA support on Wednesday to discuss the possible solutions, will keep this issue updated on the progress.

@zroubalik
Copy link
Member

@idvoretskyi any update on this please?

@idvoretskyi
Copy link
Contributor

@zroubalik still no major updates, we're speaking with GitHub on this now.

@zroubalik
Copy link
Member

@idvoretskyi any news? I am planning to disable FOSSA check on PRs if we are not able to solve this in the near future.
Thanks!

@tomkerkhove
Copy link
Member

I was thinking the same actually 😅

@idvoretskyi
Copy link
Contributor

@zroubalik I have a solution to try, let me check and I'll get back to you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request All issues for new features that have not been committed to needs-discussion
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants