Skip to content
Snippets Groups Projects

Fix issue with describing includes for child/parent pipelines

Merged Jason Yavorska requested to merge jyavorska-master-patch-41631 into master
All threads resolved!

What does this MR do?

As discussed in the issue #205157 (closed), the current documentation gives the wrong impression that trigger supports all include parameters, but it doesn't. This corrects the page to remove that line and just point back to the current trigger keyword documentation.

Related issues

Relates to #205157 (closed)

Author's checklist (required)

Do not add the feature, frontend, backend, ~"bug", or database labels if you are only updating documentation. These labels will cause the MR to be added to code verification QA issues.

When applicable:

Review checklist

All reviewers can help ensure accuracy, clarity, completeness, and adherence to the Documentation Guidelines and Style Guide.

1. Primary Reviewer

  • Review by a code reviewer or other selected colleague to confirm accuracy, clarity, and completeness. This can be skipped for minor fixes without substantive content changes.

2. Technical Writer

  • Technical writer review. If not requested for this MR, must be scheduled post-merge. To request for this MR, assign the writer listed for the applicable DevOps stage.

3. Maintainer

  1. Review by assigned maintainer, who can always request/require the above reviews. Maintainer's review can occur before or after a technical writer review.
  2. Ensure a release milestone is set.
  3. If there has not been a technical writer review, create an issue for one using the Doc Review template.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Marcel Amirault resolved all threads

    resolved all threads

  • added 1 commit

    • 83089f81 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • @jyavorska I see you assigned then unassigned me, I'm guessing this was a mistake? the changes here are good, so I'll approve and merge, let me know if that was ok! :pray:

  • Marcel Amirault approved this merge request

    approved this merge request

  • Marcel Amirault enabled an automatic merge when the pipeline for 632c3b56 succeeds

    enabled an automatic merge when the pipeline for 632c3b56 succeeds

  • Marcel Amirault mentioned in commit c86db37e

    mentioned in commit c86db37e

  • added workflowcanary label and removed workflowstaging label

  • added workflowproduction label and removed workflowcanary label

  • Marius Bobin mentioned in merge request !43404 (merged)

    mentioned in merge request !43404 (merged)

  • This merge request has been deployed to the release.gitlab.net environment, and will be included in the upcoming self-managed GitLab 13.5.0 release.


    :robot: This comment is generated automatically using the Release Tools project.

  • Please register or sign in to reply
    Loading