nasa / bplib

Apache License 2.0
27 stars 13 forks source link

Establish preliminary build/pull request validation checks #120

Closed jphickey closed 1 year ago

jphickey commented 2 years ago

There are certain items that should be confirmed with every proposed change to bplib before merging that change. Currently the list of tests is rather small but helpful nonetheless:

Both of these tests should be executed on every PR to avoid regressions.

Furthermore, additional build verification could also be added similar to the other CFS apps, to execute static analysis tools.

In the future, this would also run all coverage tests as well (but these do not yet exist for the current implementation of BPLib)

jphickey commented 1 year ago

As there are a number of workflows to be considered this can be split into it's constituent parts to be implemented individually:

Adding these can all be separately considered and implemented and thus each should be a separate issue

jphickey commented 1 year ago

Added issues #182, #183, #184, #185, #186, and #187, which will replace this.