Skip to content

Commit ce09060

Browse files
committed
Merge branch 'production' into ranbel/tunnel-onoma
2 parents f3ff54d + 389858f commit ce09060

File tree

1,460 files changed

+25048
-45671
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

1,460 files changed

+25048
-45671
lines changed

.github/CODEOWNERS

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828
/src/content/release-notes/workers-ai.yaml @kathayl @mchenco @kodster28 @cloudflare/pcx-technical-writing
2929
/src/content/release-notes/ai-gateway.yaml @kathayl @mchenco @kodster28 @cloudflare/pcx-technical-writing
3030
/src/content/release-notes/vectorize.yaml @elithrar @mchenco @sejoker @cloudflare/pcx-technical-writing
31-
/src/content/docs/autorag/ @rita3ko @irvinebroque @aninibread @ToriLindsay @cloudflare/pcx-technical-writing
31+
/src/content/docs/autorag/ @rita3ko @irvinebroque @aninibread @cloudflare/pcx-technical-writing
3232

3333
# Analytics & Logs
3434

@@ -49,8 +49,9 @@
4949

5050
# Changelogs
5151

52-
/src/content/changelog/ @cloudflare/pm-changelogs @cloudflare/pcx-cloudflare-one
53-
/src/assets/images/changelog/ @cloudflare/pm-changelogs @cloudflare/pcx-cloudflare-one
52+
/src/content/changelog/ @cloudflare/pm-changelogs @cloudflare/pcx-technical-writing
53+
/src/content/changelog/waf/ @cloudflare/firewall @vs-mg @cloudflare/pm-changelogs @cloudflare/pcx-technical-writing
54+
/src/assets/images/changelog/ @cloudflare/pm-changelogs @cloudflare/pcx-technical-writing
5455
/src/assets/images/ @cloudflare/pm-changelogs @cloudflare/pcx-technical-writing
5556

5657
# Cloudflare One
@@ -115,8 +116,8 @@
115116
/src/content/release-notes/kv.yaml @elithrar @thomasgauvin @rts-rob @oxyjun @cloudflare/pcx-technical-writing
116117
/src/content/partials/kv/ @elithrar @thomasgauvin @rts-rob @oxyjun @cloudflare/pcx-technical-writing
117118
/src/content/docs/pub-sub/ @elithrar @dcpena @cloudflare/pcx-technical-writing
118-
/src/content/docs/queues/ @elithrar @maheshwarip @harshil1712 @cloudflare/pcx-technical-writing
119-
/src/content/release-notes/queues.yaml @elithrar @maheshwarip @cloudflare/pcx-technical-writing
119+
/src/content/docs/queues/ @elithrar @jonesphillip @harshil1712 @cloudflare/pcx-technical-writing
120+
/src/content/release-notes/queues.yaml @elithrar @jonesphillip @cloudflare/pcx-technical-writing
120121
/src/content/docs/r2/ @oxyjun @elithrar @jonesphillip @aninibread @harshil1712 @cloudflare/workers-docs @cloudflare/pcx-technical-writing
121122
/src/content/release-notes/r2.yaml @oxyjun @elithrar @aninibread @cloudflare/workers-docs @cloudflare/pcx-technical-writing
122123
/src/content/docs/realtime/ @cloudflare/pcx-technical-writing @cloudflare/calls
@@ -136,7 +137,7 @@
136137
/src/content/docs/workers/frameworks/ @igorminar @cloudflare/wrangler @aninibread @GregBrimble @kodster28 @cloudflare/pcx-technical-writing
137138
/src/content/docs/pages/framework-guides/ @igorminar @cloudflare/wrangler @aninibread @GregBrimble @kodster28 @cloudflare/pcx-technical-writing
138139
/src/content/docs/analytics/analytics-engine/ @irvinebroque @elithrar @cloudflare/pcx-technical-writing
139-
/src/content/docs/cloudflare-for-platforms/workers-for-platforms/ @irvinebroque @angelampcosta @GregBrimble @cloudflare/deploy-config @cloudflare/pcx-technical-writing
140+
/src/content/docs/cloudflare-for-platforms/workers-for-platforms/ @irvinebroque @GregBrimble @cloudflare/deploy-config @cloudflare/pcx-technical-writing
140141
/src/content/docs/workers/observability/ @irvinebroque @mikenomitch @rohinlohe @kodster28 @cloudflare/pcx-technical-writing
141142
/src/content/docs/workers/static-assets @irvinebroque @GregBrimble @WalshyDev @kodster28 @cloudflare/deploy-config @cloudflare/pcx-technical-writing
142143
/src/content/docs/workflows/ @elithrar @celso @cloudflare/pcx-technical-writing
@@ -150,7 +151,6 @@
150151

151152
# Docs team areas
152153

153-
/src/content/docs/docs-guide/ @dcpena @cloudflare/pcx-technical-writing
154154
/src/content/docs/style-guide/ @dcpena @cloudflare/pcx-technical-writing
155155

156156
# Google tag gateway

.github/actions/assign-issue/index.ts renamed to .github/actions/issue-label-assign/index.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ import * as codeOwnersUtils from "codeowners-utils";
8484

8585
console.log("Assignees added (if present)");
8686

87-
// Add labels for future reporting
87+
// Add "product" labels
8888

8989
const labelPrefix = "product:";
9090
const newLabels = new Set<string>();

.github/actions/assign-issue/package-lock.json renamed to .github/actions/issue-label-assign/package-lock.json

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/actions/assign-issue/package.json renamed to .github/actions/issue-label-assign/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
22
"private": true,
33
"version": "0.0.0",
4-
"name": "assign-issue",
4+
"name": "issue-label-assign",
55
"scripts": {
66
"build": "esbuild index.ts --bundle --format=cjs --platform=node --minify --outfile=index.js"
77
},
File renamed without changes.
Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
name: Issue - Label and assign
2+
3+
# **What it does**: Labels Issues with associated products. Updates assignees to match our CODEOWNERS file.
4+
# **Why we have it**: Easier scanning of repo PRs && backend reporting on our team's work.
5+
# **Who does it impact**: PCX team
6+
7+
on:
8+
issues:
9+
types: [opened]
10+
11+
jobs:
12+
compile:
13+
name: Label and assign
14+
permissions:
15+
issues: write
16+
runs-on: ubuntu-latest
17+
steps:
18+
- uses: actions/checkout@v4
19+
- uses: ./.github/actions/issue-label-assign
20+
with:
21+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/issue.yml

Lines changed: 0 additions & 17 deletions
This file was deleted.

.github/workflows/comment-changed-filenames.yml renamed to .github/workflows/potential-redirects-or-partials.yml

Lines changed: 46 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: Flag changed filenames
1+
name: Potential redirects or partials
22

33
# **What it does**: Adds or removes a comment if a PR renames or removes a file.
44
# **Why we have it**: Highlights when we need redirects covering certain file paths.
@@ -22,13 +22,13 @@ jobs:
2222
- name: Checkout Repository
2323
uses: actions/checkout@v4
2424

25-
- name: Get Renamed or Removed Files from Pull Request
25+
- name: Potential redirects
2626
env:
2727
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
2828
run: |
2929
files=$(curl -s -H "Authorization: Bearer $GITHUB_TOKEN" \
3030
-H "Accept: application/vnd.github.v3+json" \
31-
"https://api.github.com/repos/${{ github.repository }}/pulls/${{ github.event.pull_request.number }}/files" | \
31+
"https://api.github.com/repos/${{ github.repository }}/pulls/${{ github.event.pull_request.number }}/files?per_page=100" | \
3232
jq -r '.[] | select(.status=="renamed" or .status=="removed") | select (.filename | startswith("src/content/docs")) | select(.filename | endswith(".mdx")) | if .status == "renamed" then .previous_filename else .filename end' | \
3333
sed -e 's|^src/content/docs||' -e 's|/index\.mdx$|/|' -e 's|\.mdx$|/|')
3434
# Use random delimiter for security reasons
@@ -37,17 +37,32 @@ jobs:
3737
echo "${files}" >> "$GITHUB_ENV"
3838
echo "${delimiter}" >> "$GITHUB_ENV"
3939
40+
- name: Updated partial files
41+
env:
42+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
43+
run: |
44+
files=$(curl -s -H "Authorization: Bearer $GITHUB_TOKEN" \
45+
-H "Accept: application/vnd.github.v3+json" \
46+
"https://api.github.com/repos/${{ github.repository }}/pulls/${{ github.event.pull_request.number }}/files?per_page=100" | \
47+
jq -r '.[] | select(.status=="modified") | select (.filename | startswith("src/content/partials")) | select(.filename | endswith(".mdx")) | .filename' | \
48+
sed -e 's|^src/content/partials||' -e 's|\.mdx$|/|')
49+
# Use random delimiter for security reasons
50+
delimiter="$(openssl rand -hex 8)"
51+
echo "PARTIAL_FILES<<${delimiter}" >> "$GITHUB_ENV"
52+
echo "${files}" >> "$GITHUB_ENV"
53+
echo "${delimiter}" >> "$GITHUB_ENV"
54+
4055
- name: Comment or Update Comment on PR based on changed files
4156
env:
4257
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
4358
run: |
4459
# If there are no changed files
45-
if [ -z "$CHANGED_FILES" ]; then
60+
if [ -z "$CHANGED_FILES" ] && [ -z "$PARTIAL_FILES" ]; then
4661
# Fetch the ID of the existing comment, if it exists
4762
existing_comment_id=$(curl -s -H "Authorization: Bearer $GITHUB_TOKEN" \
4863
-H "Accept: application/vnd.github.v3+json" \
4964
"https://api.github.com/repos/${{ github.repository }}/issues/${{ github.event.pull_request.number }}/comments" | \
50-
jq '.[] | select(.user.id == 41898282) | select(.body | contains("This PR changes current filenames or deletes current files")) | .id')
65+
jq '.[] | select(.user.id == 41898282) | select(.body | contains("This PR requires additional review attention because it affects the following areas")) | .id')
5166
5267
# If the comment exists, delete it
5368
if [ ! -z "$existing_comment_id" ]; then
@@ -58,18 +73,39 @@ jobs:
5873
fi
5974
else
6075
# Construct the comment body for changed files
61-
comment_body="This PR changes current filenames or deletes current files. Make sure you have [redirects](https://developers.cloudflare.com/pages/configuration/redirects/) set up to cover the following paths:"
62-
for path in $CHANGED_FILES; do
63-
clean_path=$(echo $path | sed 's/"//g') # Remove quotation marks
76+
comment_body="This PR requires additional review attention because it affects the following areas:"
77+
78+
# Add CHANGED_FILES logic
79+
if [ -n "$CHANGED_FILES" ]; then
6480
comment_body="$comment_body
81+
82+
### Redirects
83+
This PR changes current filenames or deletes current files. Make sure you have [redirects](https://developers.cloudflare.com/pages/configuration/redirects/) set up to cover the following paths:"
84+
for path in $CHANGED_FILES; do
85+
clean_path=$(echo "$path" | sed 's/"//g') # Remove quotation marks
86+
comment_body="$comment_body
6587
- [ ] \`$clean_path\`"
66-
done
88+
done
89+
fi
90+
91+
# Conditional bit for PARTIAL_FILES
92+
if [ -n "$PARTIAL_FILES" ]; then
93+
comment_body="$comment_body
94+
95+
### Partials
96+
This PR updates partial files, which are pieces of content used across multiple files in our [Render component](https://developers.cloudflare.com/style-guide/components/render/)."
97+
for path in $PARTIAL_FILES; do
98+
updated_path=$(echo "$path" | sed -e 's/"//g' -e 's/^\///' -e 's/\/$//')
99+
comment_body="$comment_body
100+
- [ ] \`$updated_path\` - [view affected files](https://developers.cloudflare.com/style-guide/components/render/?partial=$updated_path)"
101+
done
102+
fi
67103
68104
# Check if a comment already exists
69105
existing_comment_id=$(curl -s -H "Authorization: Bearer $GITHUB_TOKEN" \
70106
-H "Accept: application/vnd.github.v3+json" \
71107
"https://api.github.com/repos/${{ github.repository }}/issues/${{ github.event.pull_request.number }}/comments" | \
72-
jq '.[] | select(.user.id == 41898282) | select(.body | contains("This PR changes current filenames or deletes current files")) | .id')
108+
jq '.[] | select(.user.id == 41898282) | select(.body | contains("This PR requires additional review attention because it affects the following areas")) | .id')
73109
74110
comment_payload=$(echo '{}' | jq --arg body "$comment_body" '.body = $body')
75111

0 commit comments

Comments
 (0)