Skip to content

refactor: Convert use-local-web-components to TS#23829

Merged
Artur- merged 4 commits intomainfrom
refactor/vite-plugin-local-web-components
Mar 13, 2026
Merged

refactor: Convert use-local-web-components to TS#23829
Artur- merged 4 commits intomainfrom
refactor/vite-plugin-local-web-components

Conversation

@vursen
Copy link
Contributor

@vursen vursen commented Mar 11, 2026

Replace the plain JS module with a proper TS Vite plugin that returns a plugin config object. Also bump the Vite peer dependency to 7.x for the service-worker and local-web-components plugins.

Replace the plain JS module with a proper Vite plugin that returns a
plugin config object. Also bump the Vite peer dependency to 7.x for
the service-worker and local-web-components plugins.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vursen vursen requested a review from Artur- March 11, 2026 05:44
@vursen vursen changed the title refactor: Convert use-local-web-components to TypeScript refactor: Convert use-local-web-components to TS Mar 11, 2026
vursen and others added 2 commits March 12, 2026 14:26
Replace the plain JS module with a proper Vite plugin that returns a
plugin config object. Also bump the Vite peer dependency to 7.x for
the service-worker and local-web-components plugins.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link

github-actions bot commented Mar 12, 2026

Test Results

 1 382 files  ±0   1 382 suites  ±0   1h 25m 43s ⏱️ - 1m 6s
 9 903 tests ±0   9 832 ✅ ±0  71 💤 ±0  0 ❌ ±0 
10 376 runs  ±0  10 296 ✅ ±0  80 💤 ±0  0 ❌ ±0 

Results for commit c46e2dd. ± Comparison against base commit 6e016df.

♻️ This comment has been updated with latest results.

@sonarqubecloud
Copy link

@vursen vursen requested review from Artur- and platosha and removed request for Artur- March 12, 2026 13:09
@Artur- Artur- merged commit feaed65 into main Mar 13, 2026
32 checks passed
@Artur- Artur- deleted the refactor/vite-plugin-local-web-components branch March 13, 2026 08:33
@vaadin-bot
Copy link
Collaborator

Hi @vursen and @Artur-, when i performed cherry-pick to this commit to 25.1, i have encountered the following issue. Can you take a look and pick it manually?
Error Message:
Error: Command failed: git cherry-pick feaed65
error: could not apply feaed65... refactor: Convert use-local-web-components to TS (#23829)
hint: After resolving the conflicts, mark them with
hint: "git add/rm ", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".

vaadin-bot pushed a commit that referenced this pull request Mar 13, 2026
Replace the plain JS module with a proper TS Vite plugin that returns a plugin config object. Also bump the Vite peer dependency to 7.x for the service-worker and local-web-components plugins.
vaadin-bot added a commit that referenced this pull request Mar 13, 2026
Replace the plain JS module with a proper TS Vite plugin that returns a plugin config object. Also bump the Vite peer dependency to 7.x for the service-worker and local-web-components plugins.

Co-authored-by: Sergey Vinogradov <mr.vursen@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants