Skip to content

Commit c93b639

Browse files
authored
Merge branch 'master' into advance-rfc-0774
2 parents 430fd5a + 354adc0 commit c93b639

29 files changed

+2518
-110
lines changed

.github/actions/setup-rfcs-tooling/action.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ runs:
99
with:
1010
repository: emberjs/rfcs-tooling
1111
path: rfcs-tooling
12-
ref: 'v2.1.1'
12+
ref: 'v3.0.0'
1313

1414
- uses: actions/setup-node@v3
1515
with:

.github/workflows/advance-rfc.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ jobs:
5555
echo "A new RFC was added"
5656
echo "value=true" >> $GITHUB_OUTPUT
5757
else
58-
node rfcs-tooling/has-stage-changed.js ${{ github.event.before }} ${{ steps.rfcs.outputs.modified-rfc }}
58+
node rfcs-tooling/scripts/has-stage-changed.mjs ${{ github.event.before }} ${{ steps.rfcs.outputs.modified-rfc }}
5959
if [[ $? == 0 ]]; then
6060
echo "## Yes, stage has changed" >> $GITHUB_STEP_SUMMARY
6161
echo "value=true" >> $GITHUB_OUTPUT
@@ -69,7 +69,7 @@ jobs:
6969
if: steps.has-stage-changed.outputs.value == 'true'
7070
id: new-stage
7171
run: |
72-
new_stage=`node rfcs-tooling/find-next-stage.js ${{ steps.rfcs.outputs.modified-rfc }}`
72+
new_stage=`node rfcs-tooling/scripts/find-next-stage.mjs ${{ steps.rfcs.outputs.modified-rfc }}`
7373
echo "New Stage: $new_stage"
7474
echo "value=$new_stage" >> $GITHUB_OUTPUT
7575

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,4 +15,4 @@ jobs:
1515
uses: ./.github/actions/setup-rfcs-tooling
1616

1717
- name: Lint the frontmatter of all RFCs
18-
run: node ./rfcs-tooling/lint-rfc-frontmatter.js text/*.md
18+
run: node ./rfcs-tooling/scripts/lint-rfc-frontmatter.mjs text/*.md

.github/workflows/generate-rfc-frontmatter-json.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ jobs:
1818

1919
- name: Generate frontmatter data file
2020
run: |
21-
node rfcs-tooling/list-frontmatter.js text/*.md > rfc-data.json
21+
node rfcs-tooling/scripts/list-frontmatter.mjs text/*.md > rfc-data.json
2222
2323
- uses: actions/upload-artifact@v3
2424
with:

.github/workflows/newly-added-rfcs.yml

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ jobs:
9292

9393
- name: Verify stage of newly added RFC is `accepted` in frontmatter
9494
run: |
95-
frontmatter=`node rfcs-tooling/rfc-frontmatter.js ${{ needs.check-rfcs.outputs.modified-rfc }}`
95+
frontmatter=`node rfcs-tooling/scripts/rfc-frontmatter.mjs ${{ needs.check-rfcs.outputs.modified-rfc }}`
9696
stage=`echo $frontmatter | jq '.stage'`
9797
if [[ $stage != '"accepted"' ]]; then
9898
echo "::error::Newly added RFCs must have the stage 'accepted' in the frontmatter"
@@ -113,8 +113,7 @@ jobs:
113113
- name: Test RFC Filename matches PR Number that adds it
114114
env:
115115
PR_NUMBER: ${{ github.event.pull_request.number }}
116-
run: node check-filename-matches-pr.js $PR_NUMBER ${{ needs.check-rfcs.outputs.modified-rfc }}
117-
working-directory: rfcs-tooling
116+
run: node rfcs-tooling/scripts/check-filename-matches-pr.mjs $PR_NUMBER ${{ needs.check-rfcs.outputs.modified-rfc }}
118117

119118
check-accepted-pr-url:
120119
name: Verify Accepted PR URL is correct
@@ -129,7 +128,7 @@ jobs:
129128

130129
- name: Verify Accepted PR URL is correct
131130
run: |
132-
frontmatter=`node rfcs-tooling/rfc-frontmatter.js ${{ needs.check-rfcs.outputs.modified-rfc }}`
131+
frontmatter=`node rfcs-tooling/scripts/rfc-frontmatter.mjs ${{ needs.check-rfcs.outputs.modified-rfc }}`
133132
accepted_pr=`echo $frontmatter | jq '.prs.accepted'`
134133
accepted_pr=${accepted_pr//\"/}
135134
expected_pr="${{ github.event.pull_request.html_url }}"

.github/workflows/open-advancement-pr.yml

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,16 +28,17 @@ jobs:
2828

2929
- name: Update frontmatter
3030
run: |
31-
node rfcs-tooling/update-rfc-stage.js ${{ inputs.new-stage }} ${{ inputs.rfc-path }}
31+
node rfcs-tooling/scripts/update-rfc-stage.mjs ${{ inputs.new-stage }} ${{ inputs.rfc-path }}
3232
3333
- name: Set variables for use in PR
3434
id: pr-variables
3535
run: |
36-
frontmatter=`node rfcs-tooling/rfc-frontmatter.js ${{ inputs.rfc-path }}`
36+
frontmatter=`node rfcs-tooling/scripts/rfc-frontmatter.mjs ${{ inputs.rfc-path }}`
3737
ready_for_release_pr=`echo $frontmatter | jq '.prs."ready-for-release"'`
3838
ready_for_release_pr=${ready_for_release_pr//\"/}
3939
released_pr=`echo $frontmatter | jq '.prs.released'`
4040
released_pr=${released_pr//\"/}
41+
title=`echo $frontmatter | jq '.title'`
4142
if [[ ${{ inputs.new-stage }} == "ready-for-release" ]]; then
4243
pretty_stage="Ready for Release"
4344
template=`sed -e 's/__RFC_NUMBER__/${{ inputs.rfc-number }}/g' .github/PULL_REQUEST_TEMPLATE/advance-to-ready-for-release.md`
@@ -53,23 +54,24 @@ jobs:
5354
echo 'EOF' >> $GITHUB_OUTPUT
5455
echo "Pretty Stage: $pretty_stage"
5556
echo "pretty-stage=$pretty_stage" >> $GITHUB_OUTPUT
57+
echo "title=$title" >> $GITHUB_OUTPUT
5658
5759
- name: Open PR
5860
id: create-pr
5961
uses: peter-evans/create-pull-request@v4.2.0
6062
with:
6163
token: ${{ secrets.personal-access-token }}
62-
commit-message: "Advance RFC to Stage ${{ inputs.new-stage }}"
64+
commit-message: "Advance RFC {{ inputs.rfc-number }} to Stage ${{ inputs.new-stage }}"
6365
add-paths: 'text'
6466
branch: "advance-rfc-${{ inputs.rfc-number }}"
65-
title: "Advance RFC #${{ inputs.rfc-number}} to Stage ${{ steps.pr-variables.outputs.pretty-stage }}"
67+
title: "Advance RFC #${{ inputs.rfc-number}} `${{ steps.pr-variables.outputs.title }}` to Stage ${{ steps.pr-variables.outputs.pretty-stage }}"
6668
body: "${{ steps.pr-variables.outputs.body }}"
6769
labels: "RFC Advancement,S-${{ steps.pr-variables.outputs.pretty-stage}}"
6870
draft: true
6971

7072
- name: Add new PR link to RFC frontmatter
7173
run: |
72-
node rfcs-tooling/update-advancement-pr.js ${{ inputs.rfc-path }} ${{ inputs.new-stage}} ${{ steps.create-pr.outputs.pull-request-url }}
74+
node rfcs-tooling/scripts/update-advancement-pr.mjs ${{ inputs.rfc-path }} ${{ inputs.new-stage}} ${{ steps.create-pr.outputs.pull-request-url }}
7375
7476
- name: Update PR
7577
run: |

.github/workflows/trigger-opening-advancement-pr.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ jobs:
3333
- name: Find new stage
3434
id: new-stage
3535
run: |
36-
new_stage=`node rfcs-tooling/find-next-stage.js ${{ inputs.rfc-path }}`
36+
new_stage=`node rfcs-tooling/scripts/find-next-stage.mjs ${{ inputs.rfc-path }}`
3737
echo "New Stage: $new_stage"
3838
echo "value=$new_stage" >> $GITHUB_OUTPUT
3939

README.md

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -282,7 +282,8 @@ that RFC.
282282

283283
RFCs in [Final Comment Periods](#final-comment-periods-fcp) are labeled with
284284
[Final Comment Period](https://github.com/emberjs/rfcs/pulls?q=is%3Apr+is%3Aopen+sort%3Aupdated-desc+label%3A%22Final+Comment+Period%22)
285-
and are announced on the [Ember twitter account](https://twitter.com/emberjs).
285+
and are announced in `#news-and-announcements` on [Ember Discord](https://discord.gg/emberjs) and
286+
on the [Ember twitter account](https://twitter.com/emberjs).
286287

287288
[Quick links](#quick-links-for-pull-requests-for-proposed-and-advancing-rfcs)
288289
are provided at the top of the README to help you review what you are interested in.
@@ -450,9 +451,9 @@ further guidance.
450451

451452
For certain stage advancements, a _final comment period_ (FCP) is required. This
452453
is a period lasting 7 days. The beginning of this period will be signaled with a
453-
comment and tag on the RFC's pull request. Furthermore,
454-
[Ember's official Twitter account](https://twitter.com/emberjs) will post a
455-
tweet about the RFC to attract the community's attention.
454+
comment and tag on the RFC's pull request. Furthermore, a message will be posted in
455+
`#news-and-announcements` on [Ember Discord](https://discord.gg/emberjs) and on the
456+
[official Ember Twitter account](https://twitter.com/emberjs) to attract the community's attention.
456457

457458
An RFC can be modified based upon feedback from the [core teams] and community
458459
during the final comment period. Significant modifications may trigger a new

text/0236-deprecation-ember-string.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
stage: released
2+
stage: recommended
33
start-date: 2017-07-14T00:00:00.000Z
44
release-date: 2023-01-12T00:00:00.000Z
55
release-versions:
@@ -11,6 +11,7 @@ prs:
1111
accepted: 'https://github.com/emberjs/rfcs/pull/236'
1212
ready-for-release: 'https://github.com/emberjs/rfcs/pull/892'
1313
released: 'https://github.com/emberjs/rfcs/pull/897'
14+
recommended: 'https://github.com/emberjs/rfcs/pull/898'
1415
project-link:
1516
meta:
1617
tracking: 'https://github.com/emberjs/ember.js/issues/20340'

text/0331-deprecate-globals-resolver.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
stage: released
2+
stage: recommended
33
start-date: 2018-05-08T00:00:00.000Z
44
release-date: 2020-01-20T00:00:00.000Z
55
release-versions:

0 commit comments

Comments
 (0)