Age | Commit message (Collapse) | Author |
|
Drop zj-gitlab from the list of contributors.
|
|
Imports the Dangerfile code from GDK.
|
|
As I'm moving into a code contributor role again, I'd like to review
again too to build more knowledge of our codebase. This change suggests
zj-gitlab again to review MRs and thus I should slowly get more review
work to do.
|
|
|
|
|
|
|
|
To foster more knowledge transfer from Chris to the team, and the other
way around, chriscool is added to the list of reviewers.
|
|
Gitaly team members are maintainers by default, and as such should be
listed in the documentation. Futher, the user name is added to a Ruby
array that Danger uses to automatically apply labels.
|
|
Danger has a list of Gitaly team members, which get assigned to merge
requests and influences labeling. This change removes John from the
team.
Additionally it updates the table of maintainers in the contributing
documentation.
|
|
|
|
This will remove Jacob as suggested reviewer, and auto labeling
rules.
|
|
|
|
This will allow some automation to work properly, for example it
automatically applies labels for merge requests and such.
Also, this change adds Pavlo as a maintainer. That slipped last time.
|
|
This would be a round robin way of getting a reviewer. Minimal
implementation, to see if it works.
|
|
|
|
GitalyBot gets the issues the merge request closes and will detect the
first upcoming milestone and apply it.
This can be overridden by the user, and GitalyBot will not try again.
Another issue might be that if a MR slips, the milestone isn't updated.
|
|
Using Danger a small rule book, @GitalyBot will apply labels and have a
few more rules on the changelog.
|
|
|
|
|
|
|
|
Files got introduced by https://gitlab.com/gitlab-org/gitaly/merge_requests/719
but were missing numbers in the merge_request field.
|
|
In GitLab CE/EE, changelog entries aren't put directly into the
CHANGELOG.md but generated using `bin/changelog`. This system has been
copied and a script created to generate output once `_support/release`
is executed.
This prevents us from rebasing/merging master to remove merge conflicts.
|
|
Check if the changelog entry matches the MR title, and if the web url is
listed in the change, so no typo's can be made.
For speed we're using a slightly different container.
Fixes https://gitlab.com/gitlab-org/gitaly/issues/1176
|
|
|
|
Given failures to check simple things in the past, like if a changelog
was present, danger-gitlab is introduced to automatically perform these
tasks.
A warning was added when a changelog entry is skipped, and failures are
due to missing MR descriptions or gitaly-proto versions being incorrect.
To force diff highlighting to work, `.gitattributes` was added.
|