Skip to content

Run GitHub Actions workflow on macOS, Ubuntu and Windows#1381

Merged
orta merged 10 commits into
masterfrom
test-danger-on-windows
Sep 2, 2022
Merged

Run GitHub Actions workflow on macOS, Ubuntu and Windows#1381
orta merged 10 commits into
masterfrom
test-danger-on-windows

Conversation

@mathroule

@mathroule mathroule commented Sep 1, 2022

Copy link
Copy Markdown
Member

Description

Use GitHub Actions matrix to run checks on macOS, Ubuntu and Windows.

Changes

  • Use GitHub Actions matrix with OS
  • Cancel concurrent GitHub Actions workflow run
  • Check only pull requests to master branch
  • Reformat code

@mathroule mathroule changed the title Run CI on macOS, Ubuntu and Windows Run GitHub Actions workflow on macOS, Ubuntu and Windows Sep 1, 2022
@mathroule mathroule force-pushed the test-danger-on-windows branch from 46286d3 to 9a81090 Compare September 1, 2022 15:41
Comment thread .github/workflows/CI.yml
TOKEN+='79960c12a1e067f2ec'
DANGER_GITHUB_API_TOKEN=$TOKEN RUNNING_IN_ACTIONS=true echo 'bundle exec danger --verbose'
- name: Running the Dangerfile for this repo
if: runner.os != 'Windows'

@mathroule mathroule Sep 1, 2022

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Skip these commands on Windows, since it does not work. This should work on Windows:

          set TOKEN=7469b4e94ce21b43e3ab7a
          set TOKEN=%TOKEN%79960c12a1e067f2ec
          set DANGER_GITHUB_API_TOKEN=%TOKEN%
          set RUNNING_IN_ACTIONS=true
          bundle exec danger --verbose

By the way Danger is not properly run on CI, the command is only displayed with the echo instead of being run.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ha, oh dear. Well, this shouldn't block your PR.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix the Danger run issue in #1382.

@orta

orta commented Sep 2, 2022

Copy link
Copy Markdown
Member

OK, I've removed the webhooks for AppVeyor, so this should be some of the last builds over there

@orta orta merged commit 5ac33e3 into master Sep 2, 2022
@Kaspik Kaspik deleted the test-danger-on-windows branch September 2, 2022 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants