You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2020/04/14 23:11:58 UTC
[GitHub] [incubator-superset] craig-rueda opened a new pull request #9538:
Run CI on all pushes / PR's
craig-rueda opened a new pull request #9538: Run CI on all pushes / PR's
URL: https://github.com/apache/incubator-superset/pull/9538
### CATEGORY
Choose one
- [ ] Bug Fix
- [ ] Enhancement (new features, refinement)
- [ ] Refactor
- [x] Add tests
- [ ] Build / Development Environment
- [ ] Documentation
### SUMMARY
* Sets CI (GH Actions) to run for each push/PR
The intent of switching to GH Actions is to move away from Travis. In order to accomplish this, we need to configure the required merge checks properly for the repo as to gate merges properly. If not all CI results are required, it's possible that a failing check could slip through. If not all workflows are run for each PR, then we can't REQUIRE those which run conditionally.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [incubator-superset] ktmud commented on issue #9538: Run CI on all
pushes / PR's
Posted by GitBox <gi...@apache.org>.
ktmud commented on issue #9538: Run CI on all pushes / PR's
URL: https://github.com/apache/incubator-superset/pull/9538#issuecomment-613820630
Note that GitHub has a 20 concurrent jobs limit on free plans, and it does not auto-cancel running jobs with new pushes, etc. Was just trying to save resources by limiting the triggers. Also kind of believed the frontend and backend tests should be able to pass independently and not interfere with each other.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [incubator-superset] craig-rueda merged pull request #9538: Run CI
on all pushes / PR's
Posted by GitBox <gi...@apache.org>.
craig-rueda merged pull request #9538: Run CI on all pushes / PR's
URL: https://github.com/apache/incubator-superset/pull/9538
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [incubator-superset] ktmud edited a comment on pull request #9538: Run CI on all pushes / PR's
Posted by GitBox <gi...@apache.org>.
ktmud edited a comment on pull request #9538:
URL: https://github.com/apache/incubator-superset/pull/9538#issuecomment-613820630
Note that GitHub has a 20 concurrent jobs limit on free plans, and it does not auto-cancel running jobs with new pushes, etc. Was just trying to save resources by limiting the triggers. Also kind of believed the frontend and backend **unit** tests should be able to pass independently and not interfere with each other.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org