Codespice1 / finance_accumulator_web

GNU General Public License v2.0
0 stars 0 forks source link

Github actions #10

Closed D-Antonelli closed 4 months ago

D-Antonelli commented 4 months ago
D-Antonelli commented 4 months ago

@omonimus1 Would like to hear your opinions about the list 🙂

omonimus1 commented 4 months ago

@D-Antonelli , can you please extend on synk? You mean the profiling/performance library? :)

if so, is not needed per CI pipeline ;) For you to make it easier create the pipeline, take it as steps, which will then follow-up the same order of commans:

eslinter => Code quality check (not prettier, because prettier should be set locally); start the project run the tests upload coverage in codecov;

Then, optional, anything else; :) codeql => Static code analysis? Good to just KNOW, no need to integrate it here :)

D-Antonelli commented 4 months ago

I am getting the idea of how to automate now. Instead of intagrating everything in the beggining, it's integrated as needed - getting there!

synk is for security scanning for vulnerable dependencies etc.. (This is all I know :)). It's probably doing same thing with GitGuardian.

So, just eslint is for now Then, adding first test, integrate codecov. Afterwards others will follow (depending on need)

Thanks for clarification :)