Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MAINTENANCE] Update CODEOWNERS js rule to include front end community of practice #8763

Merged
merged 2 commits into from Sep 26, 2023

Conversation

DrewHoo
Copy link
Contributor

@DrewHoo DrewHoo commented Sep 26, 2023

  • Description of PR changes above includes a link to an existing GitHub issue
  • PR title is prefixed with one of: [BUGFIX], [FEATURE], [DOCS], [MAINTENANCE], [CONTRIB]
  • Code is linted - run invoke lint (uses black + ruff)
  • Appropriate tests and docs have been updated

For more information about contributing, see Contribute.

After you submit your PR, keep the page open and monitor the statuses of the various checks made by our continuous integration process at the bottom of the page. Please fix any issues that come up and reach out on Slack if you need help. Thanks for contributing!

@netlify
Copy link

netlify bot commented Sep 26, 2023

Deploy Preview for niobium-lead-7998 canceled.

Name Link
🔨 Latest commit f71be9b
🔍 Latest deploy log https://app.netlify.com/sites/niobium-lead-7998/deploys/6512e32ab19e4b0008c15c2d

@github-actions github-actions bot added the core label Sep 26, 2023
@ghost
Copy link

ghost commented Sep 26, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

CODEOWNERS Outdated Show resolved Hide resolved
@DrewHoo DrewHoo added this pull request to the merge queue Sep 26, 2023
Merged via the queue into develop with commit 2e23b11 Sep 26, 2023
53 checks passed
@DrewHoo DrewHoo deleted the m/update-js-codeowners-to-include-fe-cop branch September 26, 2023 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants