Skip to content
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

Move Danger.js to GitHub Action #1357

Merged
merged 4 commits into from
Aug 25, 2020
Merged

Conversation

JackuB
Copy link
Contributor

@JackuB JackuB commented Aug 21, 2020

Danger.js is having issues running on CircleCI. This will also make it a bit faster.

And rename Smoke test to Regression test
as it contains more than just smoke
@github-actions
Copy link
Contributor

github-actions bot commented Aug 21, 2020

Messages
📖

This PR will not trigger a new version. It doesn't include any commit message with feat or fix.

📖 You are modifying something in test/smoke directory, yet you are not on the branch feat/smoke-test. You can rename your branch to feat/smoke-test and Smoke tests will trigger for this PR.

Generated by 🚫 dangerJS against 2250e94

@JackuB JackuB force-pushed the chore/move-danger-to-gh-action branch 2 times, most recently from 7759c46 to af44bff Compare August 21, 2020 16:44
@JackuB JackuB force-pushed the chore/move-danger-to-gh-action branch 2 times, most recently from 1be8d0c to bdef87f Compare August 25, 2020 08:18
@JackuB JackuB force-pushed the chore/move-danger-to-gh-action branch from bdef87f to 2250e94 Compare August 25, 2020 08:24
@JackuB JackuB requested a review from a team August 25, 2020 08:28
@JackuB JackuB marked this pull request as ready for review August 25, 2020 08:28
@JackuB JackuB requested a review from a team as a code owner August 25, 2020 08:28
@ghost ghost requested review from aviadhahami and gitphill August 25, 2020 08:28
name: Danger JS
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v1
Copy link
Contributor

Choose a reason for hiding this comment

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

@JackuB , how about add a name for checkout action?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Don't have a better name for actions/checkout then Checkout ¯\_(ツ)_/¯

@JackuB JackuB merged commit 2582bda into master Aug 25, 2020
@JackuB JackuB deleted the chore/move-danger-to-gh-action branch August 25, 2020 12:05
@snyksec
Copy link

snyksec commented Aug 25, 2020

🎉 This PR is included in version 1.382.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants