Skip to content

Commit 786ac3f

Browse files
authored
chore: cleanup++ (#675)
* chore: cleanup++
1 parent a3252cf commit 786ac3f

499 files changed

Lines changed: 5111 additions & 858264 deletions

File tree

Some content is hidden

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

.changeset/config.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,5 +6,5 @@
66
"access": "restricted",
77
"baseBranch": "main",
88
"updateInternalDependencies": "patch",
9-
"ignore": ["*-example", "dotlottie-viewer"]
9+
"ignore": ["*-example", "viewer"]
1010
}

.changeset/fast-buttons-look.md

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
---
2+
"@lottiefiles/dotlottie-svelte": minor
3+
"@lottiefiles/dotlottie-react": minor
4+
"@lottiefiles/dotlottie-solid": minor
5+
"@lottiefiles/dotlottie-vue": minor
6+
"@lottiefiles/dotlottie-web": minor
7+
"@lottiefiles/dotlottie-wc": minor
8+
---
9+
10+
chore: migrate tooling to Biome, update Node/pnpm versions, and cleanup fixtures

.eslintignore

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ node_modules/
1919
# Testing
2020
coverage/
2121
__snapshots__
22-
__fixtures__
22+
fixtures
2323

2424
# Temporary or local data
2525
temp/

.eslintrc.cjs

Lines changed: 0 additions & 53 deletions
This file was deleted.
Lines changed: 34 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,44 @@
11
---
22
name: '🐛 Bug Report'
3-
about: Something isn't working
3+
about: Report a bug in dotlottie-web or one of its framework packages
4+
labels: 'bug'
45
---
56

6-
## Overview
7+
## Package
78

8-
...
9+
<!-- Which package(s) are affected? -->
910

10-
## Consuming repo
11+
- [ ] `@lottiefiles/dotlottie-web` (core)
12+
- [ ] `@lottiefiles/dotlottie-react`
13+
- [ ] `@lottiefiles/dotlottie-vue`
14+
- [ ] `@lottiefiles/dotlottie-svelte`
15+
- [ ] `@lottiefiles/dotlottie-solid`
16+
- [ ] `@lottiefiles/dotlottie-wc`
1117

12-
> What repo were you working in when this issue occurred?
18+
## Description
1319

14-
...
20+
<!-- A clear description of the bug. -->
1521

16-
## Labels
22+
## Steps to reproduce
1723

18-
- [ ] Add the `Type: Bug` label to this issue.
24+
1.
25+
2.
26+
3.
27+
28+
## Expected behavior
29+
30+
<!-- What did you expect to happen? -->
31+
32+
## Actual behavior
33+
34+
<!-- What happened instead? -->
35+
36+
## Animation file
37+
38+
<!-- If possible, attach the .lottie or .json animation file that triggers the bug. -->
39+
40+
## Environment
41+
42+
- Package version:
43+
- Browser:
44+
- OS:
Lines changed: 18 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,28 @@
11
---
22
name: '📈 Enhancement'
3-
about: Enhancement to our codebase that isn't a adding or changing a feature
3+
about: Suggest an improvement to existing functionality
4+
labels: 'enhancement'
45
---
56

6-
## Overview
7+
## Package
78

8-
...
9+
<!-- Which package(s) does this apply to? -->
910

10-
## Motivation
11+
- [ ] `@lottiefiles/dotlottie-web` (core)
12+
- [ ] `@lottiefiles/dotlottie-react`
13+
- [ ] `@lottiefiles/dotlottie-vue`
14+
- [ ] `@lottiefiles/dotlottie-svelte`
15+
- [ ] `@lottiefiles/dotlottie-solid`
16+
- [ ] `@lottiefiles/dotlottie-wc`
17+
18+
## Description
1119

12-
> What inspired this enhancement? What makes you think this should be included?
20+
<!-- What would you like to improve? -->
21+
22+
## Motivation
1323

14-
...
24+
<!-- Why is this enhancement needed? What problem does it solve? -->
1525

16-
## Labels
26+
## Proposed solution
1727

18-
- [ ] Add the `Type: Enhancement` label to this issue.
28+
<!-- If you have a specific approach in mind, describe it here. -->
Lines changed: 22 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,37 @@
11
---
22
name: '🙌 Feature Request'
3-
about: Suggest a new feature, or changes to an existing one
3+
about: Suggest a new feature or changes to an existing one
4+
labels: 'feature request'
45
---
56

6-
## Overview
7+
## Package
78

8-
...
9+
<!-- Which package(s) does this apply to? -->
10+
11+
- [ ] `@lottiefiles/dotlottie-web` (core)
12+
- [ ] `@lottiefiles/dotlottie-react`
13+
- [ ] `@lottiefiles/dotlottie-vue`
14+
- [ ] `@lottiefiles/dotlottie-svelte`
15+
- [ ] `@lottiefiles/dotlottie-solid`
16+
- [ ] `@lottiefiles/dotlottie-wc`
917

1018
## Type
1119

1220
- [ ] New feature
13-
- [ ] Changes to existing features
21+
- [ ] Changes to existing feature
22+
23+
## Description
24+
25+
<!-- Describe the feature you'd like to see. -->
1426

1527
## Motivation
1628

17-
> What inspired this feature request? What problems were you facing, or what else makes you think this should be
18-
> included?
29+
<!-- What problem does this feature solve? What use case does it enable? -->
1930

20-
...
31+
## Proposed API / usage
2132

22-
## Labels
33+
<!-- If applicable, show how this feature would be used. -->
2334

24-
- [ ] Add the `Type: Feature Request` label to this issue.
35+
```ts
36+
// Example usage
37+
```

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 16 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,25 @@
11
## Description
22

3-
<!--
4-
Please include a summary of what you want to achieve in this pull request.
3+
<!-- Summarize your changes. If this fixes an issue, include "Fixes #<issue>" below. -->
54

6-
If this fixes an existing issue, please include the issue number.
7-
-->
5+
## Package(s) affected
86

9-
## Type of change
7+
- [ ] `@lottiefiles/dotlottie-web` (core)
8+
- [ ] `@lottiefiles/dotlottie-react`
9+
- [ ] `@lottiefiles/dotlottie-vue`
10+
- [ ] `@lottiefiles/dotlottie-svelte`
11+
- [ ] `@lottiefiles/dotlottie-solid`
12+
- [ ] `@lottiefiles/dotlottie-wc`
1013

11-
<!--
12-
Remember to indicate the affected package(s), as well as the type of change for each package.
13-
-->
14+
## Type of change
1415

15-
- [ ] <!--Package Name--> Patch: Bug (non-breaking change which fixes an issue)
16-
- [ ] <!--Package Name--> Minor: New feature (non-breaking change which adds functionality)
17-
- [ ] <!--Package Name--> Major: Breaking change (fix or feature that would cause existing functionality to not work as expected)
16+
- [ ] Bug fix (non-breaking)
17+
- [ ] New feature (non-breaking)
18+
- [ ] Breaking change
19+
- [ ] Chore (build, CI, docs, refactor)
1820

1921
## Checklist
2022

21-
- [ ] This is something we need to do.
23+
- [ ] Changes have been tested locally
24+
- [ ] Tests have been added or updated
25+
- [ ] Changeset has been added (if applicable)

.github/workflows/main.yml

Lines changed: 23 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -16,29 +16,30 @@ jobs:
1616
- name: ⬇️ Checkout repo
1717
uses: actions/checkout@v4
1818

19-
- name: ⎔ Setup pnpm@v9
19+
- name: ⎔ Setup pnpm@v10
2020
uses: pnpm/action-setup@v4
2121

22-
- name: ⎔ Setup Node@v20
22+
- name: ⎔ Setup Node@v22
2323
uses: actions/setup-node@v4
2424
with:
2525
cache: pnpm
26-
node-version: 20
26+
node-version: 22
2727

2828
- name: 📥 Download deps
2929
run: pnpm install
3030

31-
- name: 🏗 Build
32-
run: pnpm build
31+
- name: 🏗 Build packages
32+
run: pnpm build:packages
33+
34+
- name: 🏗 Build apps & examples
35+
if: github.event_name == 'push' && github.ref == 'refs/heads/main'
36+
run: pnpm build:apps && pnpm build:examples
3337

3438
- name: 🔍 Verify types
3539
run: pnpm type-check
3640

37-
- name: 🕵️ Lint
38-
run: pnpm lint:prod
39-
40-
- name: 💅 Verify format (`pnpm format` committed?)
41-
run: pnpm format
41+
- name: 🕵️ Lint + Format
42+
run: pnpm exec biome check .
4243

4344
- name: 🛡️ Test (Node)
4445
run: pnpm test:node
@@ -85,6 +86,7 @@ jobs:
8586
continue-on-error: true
8687
with:
8788
github_token: ${{ secrets.GITHUB_TOKEN }}
89+
build_script: build:packages
8890

8991
npm-release:
9092
name: npm-release
@@ -111,14 +113,14 @@ jobs:
111113
# check out all commits and tags so changeset can skip duplicate tags
112114
fetch-depth: 0
113115

114-
- name: ⎔ Setup pnpm@v9
116+
- name: ⎔ Setup pnpm@v10
115117
uses: pnpm/action-setup@v4
116118

117-
- name: ⎔ Setup Node@v20
119+
- name: ⎔ Setup Node@v22
118120
uses: actions/setup-node@v4
119121
with:
120122
cache: pnpm
121-
node-version: 20
123+
node-version: 22
122124
registry-url: https://registry.npmjs.org
123125

124126
- name: 📥 Install dependencies
@@ -161,14 +163,14 @@ jobs:
161163
with:
162164
fetch-depth: 0
163165

164-
- name: ⎔ Setup pnpm@v9
166+
- name: ⎔ Setup pnpm@v10
165167
uses: pnpm/action-setup@v4
166168

167-
- name: ⎔ Setup Node@v20
169+
- name: ⎔ Setup Node@v22
168170
uses: actions/setup-node@v4
169171
with:
170172
cache: pnpm
171-
node-version: 20
173+
node-version: 22
172174
registry-url: https://npm.pkg.github.com/
173175

174176
- name: 📥 Install dependencies
@@ -239,14 +241,14 @@ jobs:
239241
with:
240242
fetch-depth: 0
241243

242-
- name: ⎔ Setup pnpm@v9
244+
- name: ⎔ Setup pnpm@v10
243245
uses: pnpm/action-setup@v4
244246

245-
- name: ⎔ Setup Node@v20
247+
- name: ⎔ Setup Node@v22
246248
uses: actions/setup-node@v4
247249
with:
248250
cache: pnpm
249-
node-version: 20
251+
node-version: 22
250252
registry-url: https://npm.pkg.github.com/
251253

252254
- name: 📥 Install dependencies
@@ -255,7 +257,7 @@ jobs:
255257
NODE_AUTH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
256258

257259
- name: 🏗 Build
258-
run: pnpm build
260+
run: pnpm build --filter='viewer...'
259261
env:
260262
NODE_AUTH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
261263

@@ -264,4 +266,4 @@ jobs:
264266
uses: peaceiris/actions-gh-pages@v4
265267
with:
266268
github_token: ${{ secrets.GITHUB_TOKEN }}
267-
publish_dir: ./apps/dotlottie-viewer/dist
269+
publish_dir: ./apps/viewer/dist

.husky/commit-msg

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

0 commit comments

Comments
 (0)