festim-dev / FESTIM

Coupled hydrogen/tritium transport and heat transfer modelling using FEniCS
https://festim.readthedocs.io/en/stable/
Apache License 2.0
90 stars 23 forks source link

Details for contribution workflow #865

Closed RemDelaporteMathurin closed 1 month ago

RemDelaporteMathurin commented 2 months ago

Proposed changes

This just adds a few steps to the contribution workflow.

Types of changes

What types of changes does your code introduce to FESTIM?

RemDelaporteMathurin commented 1 month ago

and that one should run tests locally before asking for a review etc.

Well in that case, you need an environment to contribute

codecov[bot] commented 1 month ago

Codecov Report

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

Project coverage is 99.63%. Comparing base (31864c8) to head (80ff56d). Report is 25 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #865 +/- ## ========================================== + Coverage 99.55% 99.63% +0.07% ========================================== Files 61 61 Lines 2724 2990 +266 ========================================== + Hits 2712 2979 +267 + Misses 12 11 -1 ```

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