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] Batch action don't have toml package #445

Closed
mickahell opened this issue Jul 11, 2023 · 1 comment · Fixed by #446
Closed

[BUG] Batch action don't have toml package #445

mickahell opened this issue Jul 11, 2023 · 1 comment · Fixed by #446
Assignees
Labels
bug Something isn't working priority: high High priority tasks

Comments

@mickahell
Copy link
Collaborator

Steps to reproduce the problem

Run a Batch action

see : https://github.com/qiskit-community/ecosystem/actions/runs/5518050206/jobs/10061488402
see : https://github.com/qiskit-community/ecosystem/actions/runs/5518133819/jobs/10061677988

What is the current behavior?

Action fail because he doesn't have toml package

What is the expected behavior?

Install toml package

@mickahell mickahell added the bug Something isn't working label Jul 11, 2023
@mickahell
Copy link
Collaborator Author

The reason why the ACtions submission doesn't fail is because toml is a dep of pylint and the submission is the only Action who need to run requirements-dev. (see here Collecting toml>=0.7.1 (from pylint==2.9.5->-r requirements-dev.txt (line 2)) )

The solution is just to add toml in requirements.txt

@mickahell mickahell self-assigned this Jul 11, 2023
@mickahell mickahell added the priority: high High priority tasks label Jul 11, 2023
@mickahell mickahell mentioned this issue Jul 11, 2023
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority: high High priority tasks
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant