danger-review
Passed Started
by
@mkozono
Michael Kozono
1Running with gitlab-runner 14.10.0~beta.50.g1f2fe53e (1f2fe53e)2 on green-2.shared-gitlab-org.runners-manager.gitlab.com/default 8zCxmpPt3 feature flags: FF_USE_FASTZIP:true6Using Docker executor with image registry.gitlab.com/gitlab-org/gitlab-build-images:debian-bullseye-ruby-2.7.patched-golang-1.16-git-2.33-lfs-2.9-chrome-97-node-16.14-yarn-1.22-postgresql-12-graphicsmagick-1.3.36 ...7Authenticating with credentials from job payload (GitLab Registry)8Pulling docker image registry.gitlab.com/gitlab-org/gitlab-build-images:debian-bullseye-ruby-2.7.patched-golang-1.16-git-2.33-lfs-2.9-chrome-97-node-16.14-yarn-1.22-postgresql-12-graphicsmagick-1.3.36 ...9Using docker image sha256:f5ccee577b270c6a832d9828384f62b437fcd3314314b5a98a5ab1cd248c3455 for registry.gitlab.com/gitlab-org/gitlab-build-images:debian-bullseye-ruby-2.7.patched-golang-1.16-git-2.33-lfs-2.9-chrome-97-node-16.14-yarn-1.22-postgresql-12-graphicsmagick-1.3.36 with digest registry.gitlab.com/gitlab-org/gitlab-build-images@sha256:7291ca4981a34fb65e8293b932eab1b4a4949bffa86bf4427b9f12c73f3483b7 ...11Running on runner-8zcxmppt-project-278964-concurrent-0 via runner-8zcxmppt-shared-gitlab-org-1651161627-7b27fc23...13$ eval "$CI_PRE_CLONE_SCRIPT"14Fetching changes with git depth set to 20...15Initialized empty Git repository in /builds/gitlab-org/gitlab/.git/16Created fresh repository.17remote: Enumerating objects: 100897, done. 18remote: Counting objects: 100% (100897/100897), done. 19remote: Compressing objects: 100% (72244/72244), done. 20remote: Total 100897 (delta 37993), reused 66741 (delta 24985), pack-reused 0 21Receiving objects: 100% (100897/100897), 103.82 MiB | 26.39 MiB/s, done.22Resolving deltas: 100% (37993/37993), done.24 * [new ref] refs/pipelines/527209691 -> refs/pipelines/52720969125Checking out 5267a595 as refs/merge-requests/86026/merge...26Skipping Git submodules setup28Checking cache for ruby-gems-bullseye-9-non_protected...29WARNING: file does not exist 30Failed to extract cache31Checking cache for node-modules-bullseye-test-9-non_protected...32WARNING: file does not exist 33Failed to extract cache35Using docker image sha256:f5ccee577b270c6a832d9828384f62b437fcd3314314b5a98a5ab1cd248c3455 for registry.gitlab.com/gitlab-org/gitlab-build-images:debian-bullseye-ruby-2.7.patched-golang-1.16-git-2.33-lfs-2.9-chrome-97-node-16.14-yarn-1.22-postgresql-12-graphicsmagick-1.3.36 with digest registry.gitlab.com/gitlab-org/gitlab-build-images@sha256:7291ca4981a34fb65e8293b932eab1b4a4949bffa86bf4427b9f12c73f3483b7 ...36$ source scripts/utils.sh37$ bundle_install_script "--with danger"38Successfully installed bundler-2.3.6391 gem installed40Bundler version 2.3.641production:development42Settings are listed in order of priority. The top value will be used.43clean44Set for the current user (/root/.bundle/config): true45install_flags46Set via BUNDLE_INSTALL_FLAGS: "--jobs=$(nproc) --retry=3 --quiet"47path48Set for the current user (/root/.bundle/config): "/builds/gitlab-org/gitlab/vendor"49without50Set via BUNDLE_WITHOUT: [:production, :development]51$ bundle install --jobs=$(nproc) --retry=3 --quiet --with danger && bundle check52[DEPRECATED] The `--with` flag is deprecated because it relies on being remembered across bundler invocations, which bundler will no longer do in future versions. Instead please use `bundle config set --local with 'danger'`, and stop using this flag53Don't run Bundler as root. Bundler can ask for sudo if it is needed, and54installing your bundle as root will break this application for all non-root55users on this machine.56The Gemfile's dependencies are satisfied57==> 'bundle install --jobs=$(nproc) --retry=3 --quiet --with danger && bundle check' succeeded in 553 seconds.58$ bundle pristine pg59Installing pg 1.2.3 with native extensions60==> 'bundle pristine pg' succeeded in 16 seconds.61$ run_timed_command "retry yarn install --frozen-lockfile"62$ retry yarn install --frozen-lockfile63yarn install v1.22.1764[1/5] Validating package.json...65[2/5] Resolving packages...66[3/5] Fetching packages...67[4/5] Linking dependencies...68warning "mermaid > cypress-image-snapshot@4.0.1" has unmet peer dependency "cypress@^4.5.0".69warning "@gitlab/eslint-plugin > eslint-plugin-jest > @typescript-eslint/experimental-utils > @typescript-eslint/typescript-estree > tsutils@3.17.1" has unmet peer dependency "typescript@>=2.8.0 || >= 3.2.0-dev || >= 3.3.0-dev || >= 3.4.0-dev || >= 3.5.0-dev || >= 3.6.0-dev || >= 3.6.0-beta || >= 3.7.0-dev || >= 3.7.0-beta".70warning "@graphql-eslint/eslint-plugin > graphql-config > @endemolshinegroup/cosmiconfig-typescript-loader > ts-node@9.1.1" has unmet peer dependency "typescript@>=2.7".71warning " > eslint-import-resolver-jest@3.0.2" has unmet peer dependency "eslint-plugin-import@>=1.4.0".72warning " > eslint-import-resolver-webpack@0.13.2" has unmet peer dependency "eslint-plugin-import@>=1.4.0".73[5/5] Building fresh packages...74$ node ./scripts/frontend/postinstall.js75success Dependency postinstall check passed.76Done in 67.00s.77==> 'retry yarn install --frozen-lockfile' succeeded in 67 seconds.78$ if [ -z "$DANGER_GITLAB_API_TOKEN" ]; then # collapsed multi-line command79$ bundle exec danger --fail-on-errors=true --verbose --danger_id=7feab880/builds/gitlab-org/gitlab/danger/gitaly/Dangerfile:3: warning: already initialized constant Danger::Dangerfile::TEMPLATE_MESSAGE81/builds/gitlab-org/gitlab/danger/ci_templates/Dangerfile:3: warning: previous definition of TEMPLATE_MESSAGE was here82Results:83Messages:84- [ ] **[CHANGELOG missing](https://docs.gitlab.com/ee/development/changelog.html)**:85If you want to create a changelog entry for GitLab FOSS, add the `Changelog` trailer to the commit message you want to add to the changelog.86If you want to create a changelog entry for GitLab EE, also [add the `EE: true` trailer](https://docs.gitlab.com/ee/development/changelog.html#gitlab-enterprise-changes) to your commit message.87If this merge request [doesn't need a CHANGELOG entry](https://docs.gitlab.com/ee/development/changelog.html#what-warrants-a-changelog-entry), feel free to ignore this message.88Markdown:89**If needed, you can retry the [🔁 `danger-review` job](https://gitlab.com/gitlab-org/gitlab/-/jobs/2391235548) that generated this comment.**90## Reviewer roulette91Changes that require review have been detected!92Please refer to the table below for assigning reviewers and maintainers suggested by Danger in the specified category:93| Category | Reviewer | Maintainer |94| -------- | -------- | ---------- |95| ~backend | [Catalin Irimie](https://gitlab.com/cat) (`@cat`) (UTC+1, 1 hour behind `@splattael`) | [Mikołaj Wawrzyniak](https://gitlab.com/mikolaj_wawrzyniak) (`@mikolaj_wawrzyniak`) (UTC+2, same timezone as `@splattael`) |96To spread load more evenly across eligible reviewers, Danger has picked a candidate for each97review slot, based on their timezone. Feel free to98[override these selections](https://about.gitlab.com/handbook/engineering/projects/#gitlab)99if you think someone else would be better-suited100or use the [GitLab Review Workload Dashboard](https://gitlab-org.gitlab.io/gitlab-roulette/) to find other available reviewers.101To read more on how to use the reviewer roulette, please take a look at the102[Engineering workflow](https://about.gitlab.com/handbook/engineering/workflow/#basics)103and [code review guidelines](https://docs.gitlab.com/ee/development/code_review.html).104Please consider assigning a reviewer or maintainer who is a105[domain expert](https://about.gitlab.com/handbook/engineering/projects/#gitlab) in the area of the merge request.106Once you've decided who will review this merge request, assign them as a reviewer!107Danger does not automatically notify them for you.108==> 'bundle exec danger --fail-on-errors=true --verbose --danger_id=7feab8' succeeded in 13 seconds.110Not uploading cache ruby-gems-bullseye-9-non_protected due to policy111Not uploading cache node-modules-bullseye-test-9-non_protected due to policy113Job succeeded