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

CI/CD: Linter Action #15

Merged
merged 6 commits into from
Jul 8, 2024
Merged

CI/CD: Linter Action #15

merged 6 commits into from
Jul 8, 2024

Conversation

ncvescera
Copy link
Member

@ncvescera ncvescera commented Jul 4, 2024

This pull request has added a workflow for running pylint (python linter) to identify various errors and inaccuracies within the code. This workflow allows to control every commit added inside each pull request and generates suggestions directly within the GitHub file visualization.
A Makefile was also introduced to run the pylint linter locally.

image

@ncvescera ncvescera linked an issue Jul 5, 2024 that may be closed by this pull request
4 tasks
ncvescera added 5 commits July 8, 2024 11:01
Added pylint as a development dependency and created a lint target in Makefile
for running pylint on all .py files
… pylint.yml

Created and configured new lint workflow based on Lint Action.
Changed the way Python dependencies are installed by using Poetry instead of pip,
and removed the redundant "pip install pylint" command.
Disabled creation of virtualenvs and remove outdated pip installation command
@ncvescera ncvescera changed the title ✨ (pylint): add pylint action CI/CD: Linter Action Jul 8, 2024
@ncvescera ncvescera merged commit 932f022 into main Jul 8, 2024
1 of 2 checks passed
@ncvescera ncvescera deleted the actions/pylint branch August 26, 2024 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI/CD
1 participant