-
Notifications
You must be signed in to change notification settings - Fork 45
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
Ecosystem configuration refactoring #19
Comments
bouh :p |
I think we need to address number of issue in current configuration setup:
|
Ok, for a more clear name (config_file, project_file, project_config, testing_config, ... ?)
Yes, we could make a simple function in the runner class and call it
hmm, does it really mater ? I don't know the usual best practice, but I find the json file pretty clear to read and easy to fill
Hmm.. for me if the project don't have the config file, we just should use the default parameters and that's it. I don't see the place of these parameters in the db. SuggestionMaybe we can add a check of if already exist a tox file, if yes we don't render our and just run the tests with the one who is already here |
What is the expected behavior?
Review ecosystem configuration files.
Currently configuration supports extra dependencies and tests commands and only works for python repositories and using tox as testing and configuration (which is good actually and follows other Qiskit projects)
We need at least to add extra configurable commands to run style checks and coverage.
Epic: #9
The text was updated successfully, but these errors were encountered: