Create python-package.yml #208
Conversation
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
PR Type: Enhancement
PR Summary: This pull request introduces a GitHub Actions workflow aimed at automating the process of installing Python dependencies, running tests, and linting code across multiple Python versions. It is designed to trigger on push and pull request events to the master branch. The workflow includes steps for setting up Python environments, installing dependencies, preparing data, linting with flake8, testing with pytest, checking documentation, and reporting coverage.
Decision: Comment
📝 Type: 'Enhancement' - not supported yet.
- Sourcery currently only approves 'Typo fix' PRs.
✅ Issue addressed: this change correctly addresses the issue or implements the desired feature.
No details provided.
✅ Small diff: the diff is small enough to approve with confidence.
No details provided.
General suggestions:
- Ensure that all external scripts and data files referenced in the workflow (e.g.,
download_eniric_data.sh) are available in the repository and have the correct permissions set. - Consider specifying exact versions for Python packages installed during the workflow to avoid potential inconsistencies due to package updates.
- Review and correct any typos or inaccuracies in the workflow file to prevent potential execution failures. For example, the typo in the command to install project dependencies should be corrected to match the intended package name.
- It might be beneficial to extend the workflow to include additional Python versions or to cover more branches, depending on the project's needs and compatibility requirements.
Thanks for using Sourcery. We offer it for free for open source projects and would be very grateful if you could help us grow. If you like it, would you consider sharing Sourcery on your favourite social media? ✨
| if [ -f requirements.txt ]; then pip install -r requirements.txt; fi | ||
| - name: Install eneric | ||
| run: | | ||
| pip install -e .[dev,test,docs,ci] |
There was a problem hiding this comment.
issue (llm): The command to install 'eneric' seems to have a typo. It's likely meant to be 'eniric', as referenced in later steps. This typo could lead to a failure in the workflow execution.
No description provided.