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

[BUG] Error when running action #87

Open
svallory opened this issue Nov 3, 2023 · 3 comments · Fixed by #88
Open

[BUG] Error when running action #87

svallory opened this issue Nov 3, 2023 · 3 comments · Fixed by #88
Labels
bug Something isn't working

Comments

@svallory
Copy link

svallory commented Nov 3, 2023

Describe the bug
Github complains of missing token input. There's no documentation saying that a token parameter is needed or where to get that token. Looking at the code, there's also a GITHUB_WORKSPACE env variable that I don't know how to setup

To Reproduce
Copy the code for the action from the marketplace and paste it in the workflow

@svallory svallory added the bug Something isn't working label Nov 3, 2023
@bodinsamuel
Copy link
Collaborator

bodinsamuel commented Nov 3, 2023

Hey,
Thanks for raising the issue. The repo was initially reaching Specfy's backend to upload the results. To make it usable by other tool I made it agnostic but I during the refactor I forgot the GitHub actions token, sorry about that.

I'm fixing this right now. I'm curious what is your use case if you can share? ☺️

edit: should be good now, no need to specify any arguments

bodinsamuel added a commit that referenced this issue Nov 3, 2023
Fix #87

Add more automated testing around the GitHub Actions to catch those
errors quickly.
github-actions bot pushed a commit that referenced this issue Nov 3, 2023
## [1.8.2](v1.8.1...v1.8.2) (2023-11-03)

### Bug Fixes

* github actions ([#88](#88)) ([30f421e](30f421e)), closes [#87](#87)
* **rules:** add Celery ([31bc5a8](31bc5a8))
* **rules:** add K6 ([1f09f83](1f09f83))
* **rules:** add more pkg for languages ([73139ab](73139ab))
* **rules:** add NeonDB ([5aa9b75](5aa9b75))
* **rules:** add Novu ([dd07853](dd07853))
* **rules:** add Python partial support ([7385001](7385001))
* **rules:** add Qdrant ([5bbd688](5bbd688))
* **rules:** add Selenium ([e60c287](e60c287))
* **rules:** add Zig ([38eda05](38eda05))
@bodinsamuel bodinsamuel reopened this Nov 3, 2023
@svallory
Copy link
Author

svallory commented Nov 9, 2023

I'm fixing this right now. I'm curious what is your use case if you can share?

Sorry for the delay. I was trying to use it to evaluate a project I was asked to complete.

edit: should be good now, no need to specify any arguments

Are you sure? I added my token to make it work after reading the source code

@bodinsamuel
Copy link
Collaborator

Are you sure? I added my token to make it work after reading the source code

Yes it's not required to execute the tech detection. Then if you want to use Specfy platform, you don't need anything, just go to specfy.io > connect your github org and import a repo directly from the UI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants