Skip to content
Snippets Groups Projects

GrapeDSL for variables

Merged Zeger-Jan van de Weg requested to merge zj-grapedsl-variable into master
All threads resolved!

Merge request reports

Pipeline #4588307 passed

Pipeline passed for 3d6f18ce on zj-grapedsl-variable

Test coverage 89.06% (0.01%) from 1 job
Approval is optional

Merged by avatar (Apr 23, 2025 12:52pm UTC)

Loading

Pipeline #4593999 failed

Pipeline failed for 8fe2e345 on master

Test coverage 89.06% (0.01%) from 1 job

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • username-removed-128633
  • Thanks @zj, I had a couple of notes regarding the texts (I could be wrong about them though, so feel free to discuss it!) but otherwise LGTM! :thumbsup:

  • Milestone changed to %8.13

  • Reassigned to @zj

  • Zeger-Jan van de Weg Resolved all discussions

    Resolved all discussions

  • Added 1 commit:

  • @rymai Read all the text again, this should be good now.

  • Reassigned to @rymai

  • Thanks @zj, LGTM! :heart:

  • username-removed-128633 Status changed to merged

    Status changed to merged

  • Mentioned in commit 8fe2e345

  • Mentioned in issue #22928 (moved)

  • mentioned in issue gitlab#6486

  • Please register or sign in to reply
    Loading