Skip to content

Conversation

timngyn
Copy link
Contributor

@timngyn timngyn commented Feb 3, 2023

Description of changes:

  • Add workflow to look for new files from the PR and check to see if its path is covered by CODEOWNERS. If it isn't then it adds a comment to the PR specifying which file wasn't covered by CODEOWNERS.

Related GitHub issue #, if available:

Instructions

If this PR should not be merged upon approval for any reason, please submit as a DRAFT

Which product(s) are affected by this PR (if applicable)?

  • amplify-cli
  • amplify-ui
  • amplify-studio
  • amplify-hosting
  • amplify-libraries

Which platform(s) are affected by this PR (if applicable)?

  • JS
  • iOS
  • Android
  • Flutter
  • React Native

Please add the product(s)/platform(s) affected to the PR title

Checks

  • Does this PR conform to the styleguide?

  • Does this PR include filetypes other than markdown or images? Please add or update unit tests accordingly.

  • Are any files being deleted with this PR? If so, have the needed redirects been created?

  • Are all links in MDX files using the MDX link syntax rather than HTML link syntax?

    ref: MDX: [link](https://link.com)
    HTML: <a href="https://link.com">link</a>

When this PR is ready to merge, please check the box below

  • Ready to merge

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@timngyn timngyn added the amplify/docs-engineering Issues with the docs site itself, NOT its content label Feb 3, 2023
@timngyn timngyn changed the base branch from main to add-redirects-label-workflow February 3, 2023 20:39
@timngyn timngyn marked this pull request as ready for review February 13, 2023 22:05
@timngyn timngyn requested a review from a team as a code owner February 13, 2023 22:05
@timngyn
Copy link
Contributor Author

timngyn commented Feb 13, 2023

Closing in favor of #5110

@timngyn timngyn closed this Feb 13, 2023
@timngyn timngyn deleted the add-update-codeowners-workflow branch February 13, 2023 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
amplify/docs-engineering Issues with the docs site itself, NOT its content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant