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.

Sign up for GitHub

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

Jump to bottom

separate linting and runtests pipelines #54

Merged

IsaacYangSLA merged 1 commit into master from fixes-github-app-linting

Feb 6, 2020

Contributor

wyli commented Feb 6, 2020

This PR resolves the current CI issues in #49 #52 by
decoupling the CI tasks:

running the linting job with github app (https://github.com/Project-MONAI/MONAI/actions)
running unit/integration tests with gitlab (https://gitlab.com/project-monai/MONAI/pipelines/)

(This PR is a follow-up of #19 #46 )


          separate linting and runtests

ac15415

wyli changed the title ~~separate linting and runtests pipeliines~~ separate linting and runtests pipelines

wyli requested a review from IsaacYangSLA

February 6, 2020 11:07

IsaacYangSLA approved these changes

View reviewed changes

Contributor

IsaacYangSLA left a comment

Totally agree. This also saves some CPU/GPU cycles.

IsaacYangSLA merged commit 294f6e1 into master

wyli deleted the fixes-github-app-linting branch

April 6, 2020 13:35

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet