Skip to content
This repository has been archived by the owner on Dec 20, 2023. It is now read-only.

Check CHANGELOG entries in PRs #37

Merged
merged 2 commits into from
Sep 26, 2017
Merged

Check CHANGELOG entries in PRs #37

merged 2 commits into from
Sep 26, 2017

Conversation

AliSoftware
Copy link
Collaborator

@AliSoftware AliSoftware commented Sep 26, 2017

Note that we didn't use Danger (which was the initial plan) because at the time I don't have all the rights & access for creating a clean proper bot (nice dedicated email) nor the API token. We might migrate to Danger at a later date when my IT has time to provide this 😉

@AliSoftware AliSoftware force-pushed the feature/danger branch 2 times, most recently from 1aa0d9b to 912d9d1 Compare September 26, 2017 17:29
Note: We initially wanted to integrate Danger for that, but didn't have the proper rights to create a bot and API Token yet
      so for now we did fallback to a more simple solution. We'll probably migrate to Danger later
@AliSoftware AliSoftware changed the title Add Danger to the project Check CHANGELOG entries in PRs Sep 26, 2017
@AliSoftware AliSoftware merged commit 292b9b8 into master Sep 26, 2017
@AliSoftware AliSoftware deleted the feature/danger branch September 26, 2017 17:45
@StevenWatremez StevenWatremez mentioned this pull request Mar 12, 2018
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.

1 participant