Skip to content

Added contact sales includes to CODEOWNERS

Shane Rice requested to merge srice-add-includes-to-marketing-codeowners into master

Why is this change being made?

I made a change to /source/includes/contact-sales.html.haml and @dmor pointed out that file isn't included in CODEOWNERS.

@brandon_lyon @skarpeles do y'all have any suggestions for other includes we need to add to CODEOWNERS?

cc @sbouchard1 added you as an owner, we can add other folks there too

Author Checklist

  • Provided a concise title for the MR
  • Added a description to this MR explaining the reasons for the proposed change, per say-why-not-just-what
  • Assign this change to the correct DRI
    • If the DRI for the page/s being updated isn’t immediately clear, then assign it to your manager.
    • If your manager does not have merge rights, please ask someone to merge it AFTER it has been approved by your manager in #mr-buddies.
    • If the changes relate to any part of the project other than updates to content and/or data files please make sure to ping @gl-static-site-editor in a comment for a review and merge. For example changes to .gitlab-ci.yml, JavaScript/CSS/Ruby code or the layout files.

For help with failing pipelines reach out in #mr-buddies in Slack

Edited by Shane Rice

Merge request reports