Skip to content
This repository has been archived by the owner on Jan 4, 2020. It is now read-only.

Danger #106

Closed
wants to merge 6 commits into from
Closed

Danger #106

wants to merge 6 commits into from

Conversation

cojoj
Copy link
Contributor

@cojoj cojoj commented Jun 8, 2017

Closes #102

Added Gemfile with our Ruby dependencies, updated swiftlint.yml, cause it had some issues with correctly checking files, simplified boddybuild_postclone, created very basic Dangerfile.

@cojoj
Copy link
Contributor Author

cojoj commented Jun 8, 2017

Hahaha, @JohnSundell you'll be the bad guy reporting all the bad things 😂 Seems like it works. I'll have to add more stuff to Dangerfile, but it's nice it works out of the box ❤️

@cojoj cojoj changed the title [WIP] Danger Danger Jun 8, 2017
@JohnSundell
Copy link
Owner

1 Warning
⚠️ Please provide a summary in the Pull Request description

Generated by 🚫 Danger

@JohnSundell
Copy link
Owner

Haha 😅 We should be able to configure that to say "DangerBot" or something.

@cojoj
Copy link
Contributor Author

cojoj commented Jun 8, 2017

@JohnSundell to do so I'll need access to BuddyBuild or you'll have to create Bot and add it. I have account on BuddyBuild, so I guess you can invite me, but I guess I'll be able to finish this no sooner than Tuesday (going to the wedding 👰)

@cojoj
Copy link
Contributor Author

cojoj commented Jun 13, 2017

@JohnSundell I'll kindly ask you to follow these steps and add the DANGER_GITHUB_API_TOKEN to Bot's build ENV. We shouldn't have any further changes in this PR as from now on buddybuild should handle everything correctly (Gemfile and Dangerfile are present).
We may wan to create a special doc for our guidelines, but I don't think it's necessary as Danger will handle everything and report it, so it's way more convenient and one less thing to maintain. I'd rather take a look at other projects and see some Danger checks for which we can benefit as well, but I guess I'll just propose them (if I find anything) in #102.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants