Skip to content

Commit 07f24b0

Browse files
authored
Merge branch 'ember-learn:main' into lazy-load-highcharts
2 parents 10927a1 + 2619f7e commit 07f24b0

File tree

128 files changed

+21442
-68106
lines changed

Some content is hidden

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

128 files changed

+21442
-68106
lines changed

.github/workflows/ci.yml

Lines changed: 24 additions & 39 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,10 @@ on:
66
- main
77
pull_request:
88

9+
concurrency:
10+
group: ci-${{ github.head_ref || github.ref }}
11+
cancel-in-progress: true
12+
913
env:
1014
NODE_VERSION: 20
1115
PERCY_PARALLEL_NONCE: ${{ github.run_id }}-${{ github.run_number }}
@@ -17,71 +21,52 @@ jobs:
1721
runs-on: ubuntu-latest
1822
timeout-minutes: 3
1923
steps:
20-
- name: Check out a copy of the repo
21-
uses: actions/checkout@v2
22-
23-
- uses: mansona/npm-lockfile-version@v1
24-
25-
- name: Use Node.js ${{ env.NODE_VERSION }}
26-
uses: actions/setup-node@v2
24+
- uses: actions/checkout@v4
25+
- uses: pnpm/action-setup@v4
26+
- uses: actions/setup-node@v4
2727
with:
28-
cache: 'npm'
28+
cache: 'pnpm'
2929
node-version: ${{ env.NODE_VERSION }}
30-
31-
- name: Install dependencies
32-
run: npm ci
33-
34-
- name: Lint
35-
run: npm run lint
30+
- run: pnpm i --frozen-lockfile
31+
- run: pnpm run lint
3632

3733
test-app:
3834
name: Test app
3935
runs-on: ubuntu-latest
4036
timeout-minutes: 5
4137
steps:
42-
- name: Check out a copy of the repo
43-
uses: actions/checkout@v2
44-
45-
- name: Use Node.js ${{ env.NODE_VERSION }}
46-
uses: actions/setup-node@v2
38+
- uses: actions/checkout@v4
39+
- uses: pnpm/action-setup@v4
40+
- uses: actions/setup-node@v4
4741
with:
48-
cache: 'npm'
42+
cache: 'pnpm'
4943
node-version: ${{ env.NODE_VERSION }}
50-
51-
- name: Install dependencies
52-
run: npm ci
53-
54-
- name: Test
55-
run: npx percy exec -- npm run test
44+
- run: pnpm i --frozen-lockfile
45+
- run: pnpm percy exec -- npm run test
5646
env:
5747
PERCY_PARALLEL_NONCE: ${{ env.PERCY_PARALLEL_NONCE }}
5848
PERCY_PARALLEL_TOTAL: ${{ env.PERCY_PARALLEL_TOTAL }}
59-
PERCY_TOKEN: c08aaada222e9cb103a28b98e94d71ff8baaddb5a548237432d900f1d4a9ec27
49+
PERCY_TOKEN: ${{ secrets.PERCY_TOKEN }}
6050

6151
lighthouse-ci:
6252
name: Run Lighthouse
6353
runs-on: ubuntu-latest
6454
timeout-minutes: 5
6555
steps:
66-
- name: Check out a copy of the repo
67-
uses: actions/checkout@v2
56+
- uses: actions/checkout@v4
6857
with:
6958
ref: ${{ github.event.pull_request.head.sha }}
70-
71-
- name: Use Node.js ${{ env.NODE_VERSION }}
72-
uses: actions/setup-node@v2
59+
- uses: pnpm/action-setup@v4
60+
- uses: actions/setup-node@v4
7361
with:
74-
cache: 'npm'
62+
cache: 'pnpm'
7563
node-version: ${{ env.NODE_VERSION }}
76-
77-
- name: Install dependencies
78-
run: npm ci
79-
64+
- run: pnpm i --frozen-lockfile
8065
- name: Build production app
81-
run: npm run build
66+
run: pnpm run build
8267

8368
- name: Run Lighthouse
84-
run: npx @lhci/[email protected] autorun
69+
run: pnpm lhci autorun
8570
env:
8671
LHCI_GITHUB_APP_TOKEN: ${{ secrets.LHCI_GITHUB_APP_TOKEN }}
8772
LHCI_TOKEN: ${{ secrets.LHCI_TOKEN }}

.github/workflows/ember-assets.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,10 @@ jobs:
88
permissions:
99
pull-requests: write
1010
steps:
11-
- uses: actions/checkout@v3
11+
- uses: actions/checkout@v4
1212
with:
13-
fetch-depth: 0
13+
fetch-depth: 0
14+
- uses: pnpm/action-setup@v4
1415
- uses: mainmatter/ember-asset-size-action@v3
1516
with:
1617
repo-token: "${{ secrets.GITHUB_TOKEN }}"

.netlifyredirects

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
/legal /about/legal
1313
/meetup-assets /community/meetups/assets
1414
/statusboard https://github.com/emberjs/rfc-tracking/issues
15-
/survey /survey/2022
15+
/survey /survey/2024
1616
/team /teams
1717
/tomster/* /mascots/:splat
1818
/zoey /mascots

.npmrc

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
ignore-scripts=false
2+
use-node-version=20.18.3

README.md

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -19,11 +19,11 @@ You will need the following things properly installed on your computer.
1919

2020
* `git clone <repository-url>` this repository
2121
* `cd ember-website`
22-
* `npm install`
22+
* `pnpm install`
2323

2424
## Running / Development
2525

26-
* `npm start`
26+
* `pnpm start`
2727
* Visit your app at [http://localhost:4200](http://localhost:4200).
2828
* Visit your tests at [http://localhost:4200/tests](http://localhost:4200/tests).
2929

@@ -33,17 +33,17 @@ Make use of the many generators for code, try `ember help generate` for more det
3333

3434
### Running Tests
3535

36-
* `npm test`
37-
* `npm test --server`
36+
* `pnpm test`
37+
* `pnpm test --server`
3838

3939
### Linting
4040

41-
* `npm run lint`
42-
* `npm run lint:fix`
41+
* `pnpm run lint`
42+
* `pnpm run lint:fix`
4343

4444
### Building
4545

46-
* `npm run build` (production)
46+
* `pnpm run build` (production)
4747

4848
### Deploying
4949

app/adapters/application.js

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,14 @@
11
import JSONAPIAdapter from '@ember-data/adapter/json-api';
22

33
export default class ApplicationAdapter extends JSONAPIAdapter {
4+
shouldBackgroundReloadAll() {
5+
return false;
6+
}
7+
8+
shouldBackgroundReloadRecord() {
9+
return false;
10+
}
11+
412
urlForFindAll(modelName) {
513
const path = this.pathForType(modelName);
614
return `/data/${path}/all.json`;

app/components/community-trends.hbs

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
{{! optional content, for meme / participating in the broader JS ecosystem, maintaining relevance }}
2+
{{#if (qp 'uwu')}}
3+
{{!-- lint incorrectly says this is already called out by a screen reader --}}
4+
{{!-- template-lint-disable require-valid-alt-text --}}
5+
<img
6+
class="uwu-logo"
7+
alt="ember uwu / kawaii logo (from internet trend / meme)"
8+
src="/images/ember-uwu-meme.png">
9+
{{/if}}

app/components/mascots/mascot-list/item.hbs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
<figure>
1+
<figure class="mascot-item" id={{@mascot.id}}>
22
<div class="well well-1/1">
33
<ResponsiveImage
44
@src="/images/tomsters/{{@mascot.image}}"

app/components/teams/team.hbs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
<h2
22
class="text-center"
33
data-test-field="Team Name"
4+
id={{dasherize @name}}
45
>
56
{{@name}}
67
</h2>

app/controllers/application.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import headerLinks from 'ember-styleguide/constants/links';
44
import replaceLinks from 'ember-website/utils/replace-links';
55

66
export default class ApplicationController extends Controller {
7+
queryParams = ['uwu'];
78
links = replaceLinks(headerLinks);
89
infoLinks = replaceLinks(infoLinks);
910
}

0 commit comments

Comments
 (0)