OMS-NetZero / FAIR

Finite-amplitude Impulse Response simple climate model

https://docs.fairmodel.net

Apache License 2.0

121 stars 61 forks source link

Add GitHub actions #89

Closed znicholls closed 3 years ago

znicholls commented 3 years ago

Pull request

Adds basic github actions. Things are still a bit of a mess but let's not do everything at once.

Please confirm that this pull request has done the following:

[x] Tests added (N/A)
[x] Documentation added (where applicable) (N/A)
[x] Example added (either to an existing notebook or as a new notebook, where applicable) (N/A)
[x] Description in CHANGELOG.rst added

Adding to CHANGELOG.rst

Please add a single line in the changelog notes similar to one of the following:

- (`#XX <http://link-to-pr.com>`_) Added feature which does something
- (`#XX <http://link-to-pr.com>`_) Fixed bug identified in (`#XX <http://link-to-issue.com>`_)

znicholls commented 3 years ago

@JGBroadbent we made a bit of a mess of this so this PR does more than I would like. Basically, all it does it get things setup so we can sensibly run the tests using GitHub actions (i.e. continuous integration). Things aren't set up that well yet so the tests are actually only running on my fork for now (see https://github.com/znicholls/FAIR/actions), but should be fixed soon.

@rgieseke if you're interested, your eyes are always welcome!

znicholls commented 3 years ago

Looks good to me, what changes do we need to run it away from your Fork @znicholls ?

I'm not completely sure. My hope is that once we merge, it'll be in the main repo so things will start running. If that doesn't happen, I'll dig deeper.

JGBroadbent commented 3 years ago

Ok, sounds good.

On 6 Oct 2020, at 11:47, Zeb Nicholls notifications@github.com wrote:

Looks good to me, what changes do we need to run it away from your Fork @znicholls https://github.com/znicholls ?

I'm not completely sure. My hope is that once we merge, it'll be in the main repo so things will start running. If that doesn't happen, I'll dig deeper.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/OMS-NetZero/FAIR/pull/89#issuecomment-704186904, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQR67PE62T4HBMBJ34WOF4TSJLYS3ANCNFSM4SFP5ADA.

znicholls commented 3 years ago

Going to assume silence from @njleach and @chrisroadmap means go for it so will merge this now so we can keep moving