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

Gitlab: added environment variable DANGER_SKIP_WHEN_EMPTY #1332

Merged
merged 5 commits into from Nov 1, 2022

Conversation

@ivankatliarchuk ivankatliarchuk changed the title draft: added environment variable DANGER_SKIP_WHEN_EMPTY Gitlab(drafr): added environment variable DANGER_SKIP_WHEN_EMPTY Oct 29, 2022
@ivankatliarchuk ivankatliarchuk changed the title Gitlab(drafr): added environment variable DANGER_SKIP_WHEN_EMPTY Gitlab: added environment variable DANGER_SKIP_WHEN_EMPTY Oct 29, 2022
@ivankatliarchuk
Copy link
Contributor Author

@orta would you be able to have a look?

@@ -67,7 +67,7 @@ getRuntimeCISource(app).then(source => {
if (!platform) {
console.log(chalk.red(`Could not find a source code hosting platform for ${source.name}.`))
console.log(
`Currently Danger JS only supports GitHub and BitBucket Server, if you want other platforms, consider the Ruby version or help out.`
`Platform '${source.name}' is not supported with Danger JS, if you want other platforms, consider the Ruby version or help out.`
Copy link
Member

Choose a reason for hiding this comment

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

Nice move

@orta
Copy link
Member

orta commented Nov 1, 2022

Please don't ping me because I didn't do open source in my spare time on the weekends - that said, this looks good to me, nice work.

@orta orta merged commit 0c8804f into danger:main Nov 1, 2022
@ivankatliarchuk
Copy link
Contributor Author

Sure. Apologies. I think it was Monday, but probably your are in a different time zone :-)

@ivankatliarchuk ivankatliarchuk deleted the issue_1330 branch November 2, 2022 09:59
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.

None yet

2 participants