populse / populse_mia

Multiparametric Image Analysis
Other
10 stars 9 forks source link

Get rid of Travis #328

Closed DimitriPapadopoulos closed 9 months ago

codecov[bot] commented 9 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (1a898b3) 85.19% compared to head (dda0fbd) 85.19%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #328 +/- ## ======================================= Coverage 85.19% 85.19% ======================================= Files 34 34 Lines 17823 17823 ======================================= Hits 15185 15185 Misses 2638 2638 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

servoz commented 9 months ago

@DimitriPapadopoulos What was the issue with #.travis.yml?

DimitriPapadopoulos commented 9 months ago

Why keep it? Travis has not been working fine for a long time. Besides, it started with #.

servoz commented 9 months ago

As a rule of thumb, we use the # character at the beginning of a name to indicate that it's something we no longer use, but want to keep for now...

servoz commented 9 months ago

Ok, I think we can live without this file, but please keep this little rule in mind.

DimitriPapadopoulos commented 9 months ago

That's what git is for, isn't it? But sorry about that, I can put it back if you want it there.

servoz commented 9 months ago

We can live without it. Leave it as it is! Deleting a file is not a big job, just like finding it with git.