diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS new file mode 100644 index 0000000000..af91299a34 --- /dev/null +++ b/.github/CODEOWNERS @@ -0,0 +1,30 @@ +# Give everyone ownership for everything expections listed below +* @wundergraph/Cosmo @wundergraph/Router + +# Protect the CODEOWNERS file +/.github/CODEOWNERS @StarpTech + +admission-server @JivusAyrus @StarpTech +cdn-server @JivusAyrus @StarpTech +cli @endigma @JivusAyrus @StarpTech +composition @Aenimus @JivusAyrus @StarpTech +composition-go @Aenimus @JivusAyrus @StarpTech +connect @Aenimus @JivusAyrus @StarpTech +connect-go @Aenimus @JivusAyrus @StarpTech +controlplane @JivusAyrus @wilsonrivera @StarpTech +data_migrations @JivusAyrus @wilsonrivera @StarpTech +graphqlmetrics @Noroth @StarpTech +helm @Noroth @StarpTech +infrastructure @Noroth @StarpTech +keycloak @JivusAyrus @wilsonrivera @StarpTech +otelcollector @Noroth @StarpTech +playground @wundergraph/Router @thisisnithin @StarpTech +proto @JivusAyrus @wilsonrivera @StarpTech +protographic @Noroth @StarpTech +router @Noroth @devsergiy @StarpTech +router-plugin @Noroth @StarpTech +router-tests @wundergraph/Router +shared @Aenimus @JivusAyrus @StarpTech +studio @JivusAyrus @wilsonrivera @StarpTech +# Cosmo Streams / EDFS +router/pkg/pubsub @alepane21 @StarpTech \ No newline at end of file diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index 8b46ef3467..4df1715269 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -18,12 +18,7 @@ Examples of good PR titles: - 🧹chore(deps): bump dependencies to latest versions --> -## Motivation and Context - - +@coderabbitai summary ## Checklist diff --git a/.github/workflows/pr-description.yml b/.github/workflows/pr-description.yml deleted file mode 100644 index 14b19e6fef..0000000000 --- a/.github/workflows/pr-description.yml +++ /dev/null @@ -1,54 +0,0 @@ -name: Check PR Description -on: - pull_request: - types: - - opened - - reopened - - edited - - synchronize - -permissions: - pull-requests: read - -jobs: - build_test: - runs-on: ubuntu-latest - timeout-minutes: 2 - steps: - - name: Checkout - uses: actions/checkout@v4 - - - name: Validate PR description - uses: actions/github-script@v7 - with: - script: | - const branchName = context.payload.pull_request.head.ref; - if (branchName.startsWith('release-please')) { - console.log('Skipping PR description check for release-please branch.'); - return; // Exit successfully - } - - const prDescription = context.payload.pull_request.body || ''; - const minLength = 20; // Minimum required length for PR description - - // Extract content between "## Motivation and Context" and "## Checklist" - const motivationRegex = /## Motivation and Context\s*([\s\S]*?)(?=## Checklist|$)/; - const motivationMatch = prDescription.match(motivationRegex); - const motivationContent = motivationMatch ? motivationMatch[1].trim() : ''; - - if (!motivationContent || motivationContent.length < minLength) { - core.setFailed(`The "Motivation and Context" section of your PR description is too short or missing. Please add a proper description of at least ${minLength} characters that explains the purpose of your changes. You can also use GitHub Copilot pull request summaries to help you write a good description.`); - return; - } - - // Check if description only contains comments - const contentWithoutComments = motivationContent - .replace(//g, '') // Remove HTML comments - .trim(); - - if (contentWithoutComments.length < minLength) { - core.setFailed('The "Motivation and Context" section of your PR description contains mostly comments. Please add a proper description that explains the purpose of your changes.'); - return; - } - - console.log('PR description "Motivation and Context" section is valid! ✅'); \ No newline at end of file