search

mgear-dev / mgear4

mGear v.4.x.x (python 3 ready) https://mgear4.readthedocs.io
MIT License
259 stars 91 forks source link

GitHub actions fix #385

Open chelloiaco opened 4 months ago

chelloiaco commented 4 months ago

Description of Changes

I noticed that the reason why the actions were failing was due to the fact that flask8 was looking for a folder python, which perhaps existed at some point but does not anymore. Same with Radon.

In the effort to have the actions be useful again, I added some functionality to the test to run only on Python files that have been changed (Added, Copied, Modified, Renamed). Additionally, I removed the --exit-zero argument from flask8 so we actually get a fail if the code does not meet the standards.

Let me know what you think.

Testing Done

I have tested this by temporarily adding the branch name into the branches candidates on the code_quality.yml file as seen on the commits history for this PR.

miquelcampos commented 4 months ago

Hi @chelloiaco Thanks for the PR. I will check it ASAP