search

finos / traderX

https://demo.traderx.finos.org/
Apache License 2.0
54 stars 60 forks source link

Add github action for continuous integration #222

Closed willtsai closed 1 month ago

willtsai commented 1 month ago

Per the comment in https://github.com/finos/traderX/pull/204#discussion_r1766598252, I am contributing a GitHub Action to bring continuous integration for the TraderX docker images.

netlify[bot] commented 1 month ago

Deploy Preview for lucky-concha-f3599f canceled.

Name Link
Latest commit fd849a6d3118b8cadb51c23eb4330ad6bf4dab69
Latest deploy log https://app.netlify.com/sites/lucky-concha-f3599f/deploys/66f092974ab0cb0008cb9266
willtsai commented 1 month ago

@maoo -- please give this a review, thanks!

maoo commented 1 month ago

@maoo -- please give this a review, thanks!

Spotless 😄

@DovOps - FYI, this github action is supposed - for now - to be manually executed by a TraderX maintainer; could you please review/merge this PR and then run the action? Please let @willtsai know the outcome. Thanks!

DovOps commented 1 month ago

@maoo @willtsai - I reviewed and merged this change (Thanks for putting this in!) - Action is running now.

DovOps commented 1 month ago

The action failed. I didn't catch this - but traderX is case sensitive, and in your paths it's not referenced properly. This needs fixing.