Skip to content

Commit 17b2eae

Browse files
committed
tweaks
2 parents c360a6f + 4c4f18b commit 17b2eae

File tree

10,683 files changed

+193690
-125470
lines changed

Some content is hidden

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

10,683 files changed

+193690
-125470
lines changed

.changeset/README.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
# Changesets
2+
3+
Hello and welcome! This folder has been automatically generated by `@changesets/cli`, a build tool that works
4+
with multi-package repos, or single-package repos to help you version and publish your code. You can
5+
find the full documentation for it [in our repository](https://github.com/changesets/changesets)
6+
7+
We have a quick list of common questions to get you started engaging with this project in
8+
[our documentation](https://github.com/changesets/changesets/blob/main/docs/common-questions.md)

.changeset/config.json

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
{
2+
"$schema": "https://unpkg.com/@changesets/[email protected]/schema.json",
3+
"changelog": ["@svitejs/changesets-changelog-github-compact", { "repo": "sveltejs/svelte" }],
4+
"commit": false,
5+
"fixed": [],
6+
"linked": [],
7+
"access": "public",
8+
"baseBranch": "main",
9+
"bumpVersionsWithWorkspaceProtocolOnly": true,
10+
"ignore": ["!(@sveltejs/*|svelte)"]
11+
}

.editorconfig

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,5 +11,5 @@ trim_trailing_whitespace = true
1111
[test/**/expected.css]
1212
insert_final_newline = false
1313

14-
[{package.json,.travis.yml,.eslintrc.json}]
14+
[package.json]
1515
indent_style = space

.eslintignore

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

.eslintrc.js

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

.github/ISSUE_TEMPLATE.md

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
1-
------
1+
---
22
Before filing an issue we'd appreciate it if you could take a moment to ensure
33
there isn't already an open issue or pull-request.
4-
-----
4+
---
55

66
If there's an existing issue, please add a :+1: reaction to the description of
77
the issue. One way we prioritize issues is by the number of :+1: reactions on
@@ -23,7 +23,6 @@ as you can including the following.
2323
- Svelte version (Please check you can reproduce the issue with the latest release!)
2424
- Whether your project uses Webpack or Rollup
2525

26-
- *Repeatable steps to reproduce the issue*
26+
- _Repeatable steps to reproduce the issue_
2727

28-
Thanks for being part of Svelte!
29-
-------
28+
## Thanks for being part of Svelte!

.github/ISSUE_TEMPLATE/bug_report.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
name: "\U0001F41E Bug report"
22
description: Report an issue with Svelte
3-
labels: ["triage: bug"]
3+
labels: ['triage: bug']
44
body:
55
- type: markdown
66
attributes:
@@ -26,7 +26,7 @@ body:
2626
id: logs
2727
attributes:
2828
label: Logs
29-
description: "Please include browser console and server logs around the time this bug occurred. Optional if provided reproduction. Please try not to insert an image but copy paste the log text."
29+
description: 'Please include browser console and server logs around the time this bug occurred. Optional if provided reproduction. Please try not to insert an image but copy paste the log text.'
3030
render: shell
3131
- type: textarea
3232
id: system-info

.github/ISSUE_TEMPLATE/feature_request.yml

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: "Feature Request"
1+
name: 'Feature Request'
22
description: Request a new Svelte feature
33
labels: [enhancement]
44
body:
@@ -22,13 +22,6 @@ body:
2222
placeholder: I would like to see...
2323
validations:
2424
required: true
25-
- type: textarea
26-
id: alternatives
27-
attributes:
28-
label: Alternatives considered
29-
description: "Please provide a clear and concise description of any alternative solutions or features you've considered."
30-
validations:
31-
required: true
3225
- type: dropdown
3326
id: importance
3427
attributes:

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,11 @@
11
### Before submitting the PR, please make sure you do the following
2+
23
- [ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
34
- [ ] Prefix your PR title with `feat:`, `fix:`, `chore:`, or `docs:`.
45
- [ ] This message body should clearly illustrate what problems it solves.
56
- [ ] Ideally, include a test that fails without this PR but passes with it.
7+
- [ ] If this PR changes code within `packages/svelte/src`, add a changeset (`npx changeset`).
8+
9+
### Tests and linting
610

7-
### Tests
8-
- [ ] Run the tests with `npm test` and lint the project with `npm run lint`
11+
- [ ] Run the tests with `pnpm test` and lint the project with `pnpm lint`

.github/workflows/ci.yml

Lines changed: 51 additions & 76 deletions
Original file line numberDiff line numberDiff line change
@@ -1,101 +1,76 @@
11
name: CI
22
on:
33
push:
4-
branches: [ master ]
4+
branches: [main]
55
pull_request:
66
permissions:
77
contents: read # to fetch code (actions/checkout)
8+
9+
env:
10+
# We only install Chromium manually
11+
PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD: '1'
12+
813
jobs:
9-
Setup:
10-
runs-on: ${{ matrix.os }}
11-
strategy:
12-
matrix:
13-
os: [ubuntu-latest, windows-latest, macOS-latest]
14-
steps:
15-
- uses: actions/checkout@v3
16-
- uses: actions/setup-node@v3
17-
with:
18-
node-version: 18
19-
cache: npm
20-
- run: npm install
21-
env:
22-
SKIP_PREPARE: true
23-
- run: npm run build
24-
env:
25-
PUBLISH: true
26-
- name: Upload build assets
27-
id: upload-artifact
28-
uses: actions/upload-artifact@v3
29-
with:
30-
name: build-assets
31-
path: |
32-
index.*
33-
compiler.*
34-
ssr.*
35-
action/
36-
animate/
37-
easing/
38-
internal/
39-
motion/
40-
store/
41-
transition/
42-
types/
4314
Tests:
44-
needs: Setup
4515
runs-on: ${{ matrix.os }}
4616
timeout-minutes: 15
4717
strategy:
4818
matrix:
49-
node-version: [8, 10, 12, 14, 16, 18]
50-
os: [ubuntu-latest, windows-latest, macOS-latest]
19+
include:
20+
- node-version: 18
21+
os: windows-latest
22+
- node-version: 18
23+
os: macOS-latest
24+
- node-version: 18
25+
os: ubuntu-latest
26+
- node-version: 20
27+
os: ubuntu-latest
28+
- node-version: 22
29+
os: ubuntu-latest
30+
5131
steps:
52-
- uses: actions/checkout@v3
53-
- uses: actions/setup-node@v3
32+
- uses: actions/checkout@v4
33+
- uses: pnpm/action-setup@v4
34+
- uses: actions/setup-node@v4
5435
with:
5536
node-version: ${{ matrix.node-version }}
56-
cache: npm
57-
- name: Download build assets
58-
uses: actions/download-artifact@v3
59-
id: download-artifact
60-
with:
61-
name: build-assets
62-
- name: Get Node version ${{ runner.os }}
63-
run: echo "NODE_VERSION=`node --version`" >> $GITHUB_ENV
64-
if: runner.os != 'Windows'
65-
- name: Get Node version ${{ runner.os }}
66-
run: |
67-
chcp 65001
68-
echo ("NODE_VERSION=$(node --version)") >> $env:GITHUB_ENV
69-
if: runner.os == 'Windows'
70-
- run: npm install --save-dev puppeteer@13
71-
if: ${{ runner.os == 'Linux' && (!startsWith(env.NODE_VERSION, 'v8.') && !startsWith(env.NODE_VERSION, 'v10.')) }}
72-
- run: npm install
73-
env:
74-
SKIP_PREPARE: true
75-
- run: npm run test:integration
37+
cache: pnpm
38+
- run: pnpm install --frozen-lockfile
39+
- run: pnpm playwright install chromium
40+
- run: pnpm test
7641
env:
7742
CI: true
7843
Lint:
7944
runs-on: ubuntu-latest
8045
timeout-minutes: 5
8146
steps:
82-
- uses: actions/checkout@v3
83-
- uses: actions/setup-node@v3
47+
- uses: actions/checkout@v4
48+
- uses: pnpm/action-setup@v4
49+
- uses: actions/setup-node@v4
8450
with:
85-
cache: npm
86-
- run: 'npm i && npm run lint'
87-
Unit:
88-
runs-on: ${{ matrix.os }}
89-
timeout-minutes: 10
90-
strategy:
91-
matrix:
92-
os: [ubuntu-latest, windows-latest, macOS-latest]
51+
node-version: 18
52+
cache: pnpm
53+
- name: install
54+
run: pnpm install --frozen-lockfile
55+
- name: type check
56+
run: pnpm check
57+
- name: lint
58+
if: (${{ success() }} || ${{ failure() }}) # ensures this step runs even if previous steps fail (avoids multiple runs uncovering different issues at different steps)
59+
run: pnpm lint
60+
- name: build and check generated types
61+
if: (${{ success() }} || ${{ failure() }}) # ensures this step runs even if previous steps fail
62+
run: pnpm build && { [ "`git status --porcelain=v1`" == "" ] || (echo "Generated types have changed — please regenerate types locally and commit the changes after you have reviewed them"; git diff; exit 1); }
63+
Benchmarks:
64+
runs-on: ubuntu-latest
65+
timeout-minutes: 15
9366
steps:
94-
- uses: actions/checkout@v3
95-
- uses: actions/setup-node@v3
67+
- uses: actions/checkout@v4
68+
- uses: pnpm/action-setup@v4
69+
- uses: actions/setup-node@v4
9670
with:
97-
cache: npm
98-
- run: npm install
71+
node-version: 18
72+
cache: pnpm
73+
- run: pnpm install --frozen-lockfile
74+
- run: pnpm bench
9975
env:
100-
SKIP_PREPARE: true
101-
- run: npm run test:unit
76+
CI: true

0 commit comments

Comments
 (0)