search

digital-asset / ex-bond-trading

A simple bond trading application
Apache License 2.0
16 stars 13 forks source link

Build forks? #9

Closed jordan-da closed 4 years ago

jordan-da commented 5 years ago

With the current setup, external users need to PR with forks, but forks aren't built and thus, are never tested and therefore, can never be merged

SO.. only PRs from internal contributors can be merged... which is an issue

cocreature commented 4 years ago

That’s no longer true for the current CI setup afaik.

dasormeter commented 4 years ago

builds were not enabled on forks : i just enabled them