ESA-APEx / apex_algorithms

Hosted APEx algorithms
Apache License 2.0
0 stars 1 forks source link

Require/enforce "main" or "permalink" algorithm references #48

Open soxofaan opened 2 days ago

soxofaan commented 2 days ago

Observation from reviewing https://github.com/ESA-APEx/apex_algorithms/pull/44: some references use feature branch references: e.g.

note that both of these contain refs/heads/hv_pv_detection which means that they track the feature branch hv_pv_detection of the PR itself. These references will break once the hv_pv_detection feature branch is removed.

Can/should we enforce pinned references (called "permalink" in the github UI) and/or main branch references? This could be handled as pre-commit things and in unit tests

HansVRP commented 2 hours ago

It is also somewhat inconvenient for entry users to retrieve such a link. For future versions it could be useful to explore how this could be facilitated to automated.