Skip to content
Snippets Groups Projects

Update the CONTRIBUTING.md docs and adjust wording in a few other places

Remove the version property from the compose file to stop having the warning printed, since that property is obsolete.

Edited by 🤖 GitLab Bot 🤖

Merge request reports

Approved by

Merged by Patrick RicePatrick Rice 1 month ago (Jan 23, 2025 8:02pm UTC)

Merge details

  • Changes merged into main with f61ae5f4 (commits were squashed).
  • Deleted the source branch.
  • Auto-merge enabled

Pipeline #1638843085 failed

Pipeline failed for f61ae5f4 on main

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Jeremy Tymes
  • Jeremy Tymes added 1 commit

    added 1 commit

    • 45eea7ea - Update the CONTRIBUTING.md docs and adjust wording in a few other places

    Compare with previous version

  • Patrick Rice approved this merge request

    approved this merge request

  • Patrick Rice enabled automatic add to merge train when checks pass

    enabled automatic add to merge train when checks pass

  • Patrick Rice resolved all threads

    resolved all threads

  • 1 Warning
    :warning: This merge request does not refer to an existing milestone.

    Reviewer roulette

    Changes that require review have been detected! A merge request is normally reviewed by both a reviewer and a maintainer in its primary category and by a maintainer in all other categories.

    To spread load more evenly across eligible reviewers, Danger has picked a candidate for each review slot. Feel free to override these selections if you think someone else would be better-suited or use the GitLab Review Workload Dashboard to find other available reviewers.

    To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines. Please consider assigning a reviewer or maintainer who is a domain expert in the area of the merge request.

    Once you've decided who will review this merge request, mention them as you normally would! Danger does not automatically notify them for you.

    Reviewer Maintainer
    @timofurrer profile link current availability (UTC+1) @patrickrice profile link current availability (UTC+0)

    If needed, you can retry the :repeat: danger-review job that generated this comment.

    Generated by :no_entry_sign: Danger

  • merged

  • Patrick Rice mentioned in commit f61ae5f4

    mentioned in commit f61ae5f4

  • @jtymes, how was your code review experience with this merge request? Please tell us how we can continue to iterate and improve:

    1. React with a :thumbsup: or a :thumbsdown: on this comment to describe your experience.
    2. Create a new comment starting with @gitlab-bot feedback below, and leave any additional feedback you have for us in the comment.

    As a benefit of being a GitLab Community Contributor, you have access to GitLab Duo, our AI-powered features. With Code Suggestions, Chat, Root Cause Analysis and more AI-powered features, GitLab Duo helps to boost your efficiency and effectiveness by reducing the time required to write and understand code and pipelines. Visit the GitLab Duo documentation to learn more about the benefits.

    Subscribe to the GitLab Community Newsletter for contributor-focused content and opportunities to level up.

    Thanks for your help! :heart:

    This message was generated automatically. Improve it or delete it.

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading