danger-review
Passed Started
by
@divyamtayal
Divyam Tayal
215:28:29 on blue-2.shared-gitlab-org.runners-manager.gitlab.com/default NL4gfoBe, system ID: s_74c3e1316164615:28:29Using Docker executor with image registry.gitlab.com/gitlab-org/gitlab-build-images/debian-bookworm-ruby-3.2.5-golang-1.23-rust-1.73-node-20.12-postgresql-14:rubygems-3.5-git-2.45-lfs-2.9-chrome-123-yarn-1.22-graphicsmagick-1.3.36 ...815:28:31Pulling docker image registry.gitlab.com/gitlab-org/gitlab-build-images/debian-bookworm-ruby-3.2.5-golang-1.23-rust-1.73-node-20.12-postgresql-14:rubygems-3.5-git-2.45-lfs-2.9-chrome-123-yarn-1.22-graphicsmagick-1.3.36 ...915:28:32Using docker image sha256:98fbc81a12d87667ce60120dd25744f54c496f6473fb807286a0eed60050a2c3 for registry.gitlab.com/gitlab-org/gitlab-build-images/debian-bookworm-ruby-3.2.5-golang-1.23-rust-1.73-node-20.12-postgresql-14:rubygems-3.5-git-2.45-lfs-2.9-chrome-123-yarn-1.22-graphicsmagick-1.3.36 with digest registry.gitlab.com/gitlab-org/gitlab-build-images/debian-bookworm-ruby-3.2.5-golang-1.23-rust-1.73-node-20.12-postgresql-14@sha256:ec1ca5818bb716477f2e333565cfae64abd56d24ee04cb09fc2cc7c9be577dc1 ...1115:28:32Running on runner-nl4gfobe-project-41372369-concurrent-0 via runner-nl4gfobe-shared-gitlab-org-1730557994-1613f3b3...2115:29:09Downloading cache from https://storage.googleapis.com/gitlab-com-runners-cache/project/41372369/ruby-gems-debian-bookworm-ruby-3.2.5-gemfile-Gemfile ETag="c261d13a5d00c72d8fcd2eb9ce0c06d0"2415:29:21Downloading cache from https://storage.googleapis.com/gitlab-com-runners-cache/project/41372369/node-modules-debian-bookworm-test ETag="b5174a87993fac2cebaa3bb25dd024c1"2715:29:47Using docker image sha256:98fbc81a12d87667ce60120dd25744f54c496f6473fb807286a0eed60050a2c3 for registry.gitlab.com/gitlab-org/gitlab-build-images/debian-bookworm-ruby-3.2.5-golang-1.23-rust-1.73-node-20.12-postgresql-14:rubygems-3.5-git-2.45-lfs-2.9-chrome-123-yarn-1.22-graphicsmagick-1.3.36 with digest registry.gitlab.com/gitlab-org/gitlab-build-images/debian-bookworm-ruby-3.2.5-golang-1.23-rust-1.73-node-20.12-postgresql-14@sha256:ec1ca5818bb716477f2e333565cfae64abd56d24ee04cb09fc2cc7c9be577dc1 ...12015:32:56$ if [[ "$CI_MERGE_REQUEST_PROJECT_URL" != "$CI_MERGE_REQUEST_SOURCE_PROJECT_URL" || -z "${GITLAB_API_TOKEN}" ]]; then # collapsed multi-line command12515:33:05Importing plugin at /builds/gitlab-community/gitlab/danger/plugins/analytics_instrumentation.rb12615:33:05Importing plugin at /builds/gitlab-community/gitlab/danger/plugins/bulk_database_actions.rb12715:33:05Importing plugin at /builds/gitlab-community/gitlab/danger/plugins/change_column_default.rb13415:33:05Importing plugin at /builds/gitlab-community/gitlab/danger/plugins/database_dictionary.rb13815:33:05Importing plugin at /builds/gitlab-community/gitlab/danger/plugins/gitlab_schema_validation.rb13915:33:05Importing plugin at /builds/gitlab-community/gitlab/danger/plugins/ignored_model_columns.rb14015:33:05Importing plugin at /builds/gitlab-community/gitlab/danger/plugins/master_pipeline_status.rb15315:33:05Importing rule analytics_instrumentation at /builds/gitlab-community/gitlab/danger/analytics_instrumentation15515:33:05Importing rule bulk_database_actions at /builds/gitlab-community/gitlab/danger/bulk_database_actions15615:33:05Importing rule ce_ee_vue_templates at /builds/gitlab-community/gitlab/danger/ce_ee_vue_templates15715:33:05Importing rule change_column_default at /builds/gitlab-community/gitlab/danger/change_column_default15815:33:05Importing rule changelog at /builds/gitlab-community/gitlab/vendor/ruby/3.2.0/gems/gitlab-dangerfiles-4.8.0/lib/danger/rules/changelog15915:33:05Importing rule changes_size at /builds/gitlab-community/gitlab/vendor/ruby/3.2.0/gems/gitlab-dangerfiles-4.8.0/lib/danger/rules/changes_size16415:33:06Importing rule commit_messages at /builds/gitlab-community/gitlab/vendor/ruby/3.2.0/gems/gitlab-dangerfiles-4.8.0/lib/danger/rules/commit_messages16515:33:10Importing rule commits_counter at /builds/gitlab-community/gitlab/vendor/ruby/3.2.0/gems/gitlab-dangerfiles-4.8.0/lib/danger/rules/commits_counter16815:33:10Importing rule customer_success at /builds/gitlab-community/gitlab/danger/customer_success17215:33:10Importing rule duplicate_yarn_dependencies at /builds/gitlab-community/gitlab/danger/duplicate_yarn_dependencies17715:33:10Importing rule gitlab_schema_validation at /builds/gitlab-community/gitlab/danger/gitlab_schema_validation17815:33:10Importing rule ignored_model_columns at /builds/gitlab-community/gitlab/danger/ignored_model_columns17915:33:10Importing rule master_pipeline_status at /builds/gitlab-community/gitlab/danger/master_pipeline_status18015:33:10Importing rule metadata at /builds/gitlab-community/gitlab/vendor/ruby/3.2.0/gems/gitlab-dangerfiles-4.8.0/lib/danger/rules/metadata18115:33:10Importing rule model_validations at /builds/gitlab-community/gitlab/danger/model_validations19015:33:10Importing rule settings_sections at /builds/gitlab-community/gitlab/danger/settings_sections19315:33:10Importing rule specialization_labels at /builds/gitlab-community/gitlab/danger/specialization_labels19515:33:10Importing rule stable_branch_patch at /builds/gitlab-community/gitlab/danger/stable_branch_patch19715:33:10Importing rule type_label at /builds/gitlab-community/gitlab/vendor/ruby/3.2.0/gems/gitlab-dangerfiles-4.8.0/lib/danger/rules/type_label19915:33:10Importing rule vue_shared_documentation at /builds/gitlab-community/gitlab/danger/vue_shared_documentation20015:33:10Importing rule z_add_labels at /builds/gitlab-community/gitlab/vendor/ruby/3.2.0/gems/gitlab-dangerfiles-4.8.0/lib/danger/rules/z_add_labels20115:33:10Importing rule z_retry_link at /builds/gitlab-community/gitlab/vendor/ruby/3.2.0/gems/gitlab-dangerfiles-4.8.0/lib/danger/rules/z_retry_link20415:33:10- [ ] 180f2ae6406086925997c15bc7f7205f72a230f0: The commit subject must not end with a period. For more information, take a look at our [Commit message guidelines](https://docs.gitlab.com/ee/development/contributing/merge_request_workflow.html#commit-messages-guidelines).20515:33:10- [ ] 91746f7c57832c820c426e68afb3ead181fbb794: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our [Commit message guidelines](https://docs.gitlab.com/ee/development/contributing/merge_request_workflow.html#commit-messages-guidelines).20815:33:10If this merge request needs a changelog entry, add the `Changelog` trailer to the commit message you want to add to the changelog.20915:33:10If 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.21415:33:10| ~"backend" | `@alexbuijs` [{width=12}](https://gitlab.com/alexbuijs) {width=12} (UTC+1) | `@sashi_kumar` [{width=12}](https://gitlab.com/sashi_kumar) {width=12} (UTC+0) |21515:33:10| ~"frontend" | `@nradina` [{width=12}](https://gitlab.com/nradina) {width=12} (UTC+1) | `@vitallium` [{width=12}](https://gitlab.com/vitallium) {width=12} (UTC+1) |21615:33:10| ~test for `spec/features/*` | `@hmuralidhar` [{width=12}](https://gitlab.com/hmuralidhar) {width=12} (UTC+11) | Maintainer review is optional for ~test for `spec/features/*` |21715:33:10| ~UX | `@pedroms` [{width=12}](https://gitlab.com/pedroms) {width=12} (UTC+0) | Maintainer review is optional for ~UX |21815:33:10| ~"group::authentication" | Reviewer review is optional for ~"group::authentication" | `@adil.farrukh` [](https://gitlab.com/adil.farrukh) |21915:33:10Please refer to [documentation page](https://docs.gitlab.com/ee/development/code_review.html#reviewer-roulette)22315:33:10**If needed, you can retry the [🔁 `danger-review` job](https://gitlab.com/gitlab-community/gitlab/-/jobs/8253374231) that generated this comment.** 
