-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Set up Continuous Integration #74
Comments
We do have Travis-CI setup already (https://travis-ci.org/github/apprenticelearner/AL_Core), which also connects to Coveralls (https://coveralls.io/github/apprenticelearner/AL_Core). Happy to use something else if someone wants to take point on setting it up. |
Okay I must have forgotten about this. Closing. |
I think we can set that up through branch protection rules. I've
contemplated kickin those on to at least prevent direct commits to master
without going through a PR first. I'll look into setting up the status
check system.
…On Wed, Dec 9, 2020 at 11:22 AM DannyWeitekamp ***@***.***> wrote:
Okay I must have forgotten about this. Closing.
Reopen if there are there relevant todos left on this.
For example something I've encountered in other projects that is nice is
the ability for people issuing PRs to run the CI and have the results show
up in the PR chat. But we don't have enough tests that they couldn't be run
locally so that isn't a big advantage at this point.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#74 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAIADH4EOX65ZWKBKHH3EYLST6P4FANCNFSM4USR4RFQ>
.
--
Erik Harpstead
Systems Scientist
Human-Computer Interaction Institute
Carnegie Mellon University
harpstead@cmu.edu
|
Azure pipelines is probably the best choice for this. It is free for open-source projects.
The text was updated successfully, but these errors were encountered: