Skip to content

Run some tests on GitHub Actions CI #8312

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
Merged
merged 1 commit into from
Feb 14, 2020
Merged

Run some tests on GitHub Actions CI #8312

merged 1 commit into from
Feb 14, 2020

Conversation

anatoliykmetyuk
Copy link
Contributor

No description provided.

@anatoliykmetyuk anatoliykmetyuk self-assigned this Feb 13, 2020
@anatoliykmetyuk
Copy link
Contributor Author

anatoliykmetyuk commented Feb 13, 2020 •
edited
Loading

We can do the transition in several steps:

  1. (This PR): Run test, test_bootstrapped and community_build on each push. Also run sbt and java11 tests on tags (probably won't happen until next release). These tests will run on GitHub Actions machine, not ours. Hence this PR won't affect the performance or semantics of the existing CI. We'll be able to see if GH Actions does the right thing at the Actions tab. This PR's CI run is available on Staging only.
  2. Release nightly builds from GitHub Actions. Also, generate the documentation for them.
  3. Make Dotty releases a responsibility of GitHub Actions, transition their execution on our machines and ditch Drone CI altogether.

@anatoliykmetyuk anatoliykmetyuk linked an issue Feb 13, 2020 that may be closed by this pull request
Drone CI license limit exceeded #8272
Closed
@anatoliykmetyuk anatoliykmetyuk requested review from nicolasstucki and removed request for nicolasstucki February 13, 2020 15:55
@nicolasstucki
Copy link
Contributor

It looks like the community build workflow has an issue. https://github.com/dotty-staging/dotty/runs/445501828?check_suite_focus=true

@nicolasstucki
Copy link
Contributor

Test output contains

[>                                      ] completed (53/1080, 0 failed, 10s)
[=>                                     ] completed (54/1080, 0 failed, 10s)
[=>                                     ] completed (54/1080, 0 failed, 11s)
[=>                                     ] completed (55/1080, 0 failed, 11s)
[=>                                     ] completed (56/1080, 0 failed, 11s)
[=>                                     ] completed (57/1080, 0 failed, 11s)
[=>                                     ] completed (58/1080, 0 failed, 11s)
[=>                                     ] completed (59/1080, 0 failed, 12s)
[=>                                     ] completed (59/1080, 0 failed, 12s)
[=>                                     ] completed (60/1080, 0 failed, 12s)
[=>                                     ] completed (62/1080, 0 failed, 12s)
[=>                                     ] completed (63/1080, 0 failed, 12s)
[=>                                     ] completed (63/1080, 0 failed, 13s)
[=>                                     ] completed (63/1080, 0 failed, 13s)
[=>                                     ] completed (64/1080, 0 failed, 13s)

we need to change the line width.

nicolasstucki
nicolasstucki reviewed Feb 14, 2020
View reviewed changes
@anatoliykmetyuk
Copy link
Contributor Author

So far community build is not included in the ci, needs a bit of debugging

@nicolasstucki nicolasstucki merged commit 5374d91 into scala:master Feb 14, 2020
@nicolasstucki nicolasstucki deleted the actions-test branch February 14, 2020 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nicolasstucki nicolasstucki nicolasstucki approved these changes

Assignees

@nicolasstucki nicolasstucki

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Drone CI license limit exceeded
2 participants
@anatoliykmetyuk @nicolasstucki