search

Exo-TiC / ExoTiC-ISM

This is a repository for the reduction pipeline detailed in Wakeford, et al., 2016, ApJ. The method implements marginalization across a series of models to represent stochastic models for observatory and instrument systematics. This is primarily for HST WFC3, however, may be extended to STIS in the future.
MIT License
8 stars 6 forks source link

Incorporate CI with GitHub Actions #87

Closed ivalaginja closed 4 years ago

ivalaginja commented 4 years ago

Replacement for #86 since that one was forcibly deleted while working on #49.

Working on issue #28 to set up CI. Since we have an environment.yml file to use with Conda, I used this open PR to GitHub Actions starter-workflows as a template: https://github.com/actions/starter-workflows/pull/197/files

I also used this link for rough guidance: https://medium.com/swlh/automate-python-testing-with-github-actions-7926b5d8a865

I adapted the package name and the top part where it defines when it should run the test, setting it to run a test each time there is an update to a PR going into master or develop. And I reduced the python versions to 3.6 and 3.7 only.

ivalaginja commented 4 years ago

Aaaand we have passing tests. In later work, we can focus on writing more extensive and more detailed tests, as well as defining the testing suite better (which flake8 tests, what OS, etc).

We also might consider to change the settings of the repository such that merging is only allowed if all tests pass (this will still allow overwriting by an admin) @hrwakeford