ome / omero-test-infra

Test infrastructure for decoupled repositories
BSD 2-Clause "Simplified" License
3 stars 13 forks source link

Check .omeroci/env for env-vars #42

Closed manics closed 5 years ago

manics commented 5 years ago

This is useful where environment variables always need to be set, regardless of the test environment. For instance https://github.com/ome/omero-metadata/blob/v0.4.1/.travis.yml#L5 always requires PLUGIN=metadata to be set including when run locally, so it should be automatic and not require additional setup by the developer.

sbesson commented 5 years ago

Generally, looks like a very neat feature simplifying the case of downstream consumers like omero-metadata. Can you also update the README to document the new extension point? Then LGTM

manics commented 5 years ago

Updated