Skip to content

deps: Update deps and move to yarn #72

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
thelukewalton merged 2 commits into from
Feb 19, 2026
Merged

deps: Update deps and move to yarn #72

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
e1f4a9d
deps: Update deps and move to yarn
thelukewalton Feb 19, 2026
7d7ff7d
[automated commit] lint format and import sort
invalid-email-address Feb 19, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/CODEOWNERS
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
* @thelukewalton
Copy link

Copilot AI Feb 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The CODEOWNERS file entry is missing the '@' prefix. GitHub CODEOWNERS requires usernames to be prefixed with '@'. This should be * @thelukewalton instead of * thelukewalton.

Copilot uses AI. Check for mistakes.
8 changes: 3 additions & 5 deletions .github/workflows/check-dist.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,15 +35,13 @@ jobs:
uses: actions/setup-node@v6
with:
node-version-file: .node-version
cache: npm

- name: Install Dependencies
id: install
run: npm ci
- name: Install dependencies
run: yarn install --immutable

- name: Build dist/ Directory
id: build
run: npm run bundle
run: yarn bundle

# This will fail the workflow if the `dist/` directory is different than
# expected.
Expand Down
5 changes: 2 additions & 3 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,10 +24,9 @@ jobs:
uses: actions/setup-node@v6
with:
node-version-file: .node-version
Copy link

Copilot AI Feb 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The removal of cache: npm from the setup-node action is correct since the project is now using Yarn. However, to maintain caching benefits, consider adding cache: yarn to improve CI performance by caching Yarn dependencies.

Suggested change
node-version-file: .node-version
node-version-file: .node-version
cache: yarn

Copilot uses AI. Check for mistakes.
cache: npm

- name: Install Dependencies
run: npm ci
- name: Install dependencies
run: yarn install --immutable

- name: Code Quality
uses: ./
Expand Down
Binary file added .yarn/install-state.gz
View file
Open in desktop
Binary file not shown.
942 changes: 942 additions & 0 deletions .yarn/releases/yarn-4.12.0.cjs
View file
Open in desktop

Large diffs are not rendered by default.

3 changes: 3 additions & 0 deletions .yarnrc.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
nodeLinker: node-modules

yarnPath: .yarn/releases/yarn-4.12.0.cjs
14 changes: 7 additions & 7 deletions custom-elements.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,19 +8,19 @@
"declarations": [
{
"kind": "variable",
"name": "maxSockets",
"type": {
"text": "number"
},
"default": "100"
"name": "headers"
},
{
"kind": "variable",
"name": "headers"
"name": "request"
},
{
"kind": "variable",
"name": "request"
"name": "maxSockets",
"type": {
"text": "number"
},
"default": "100"
},
{
"kind": "function",
Expand Down
Loading