sverweij / dependency-cruiser

Validate and visualize dependencies. Your rules. JavaScript, TypeScript, CoffeeScript. ES6, CommonJS, AMD.
https://npmjs.com/dependency-cruiser
MIT License
5.15k stars 249 forks source link

feat: support svelte 4 #948

Closed kubotak-is closed 1 month ago

kubotak-is commented 1 month ago

Description

This pull request adds support for Svelte version 4. While Svelte's next version will be 5, this pull request provides support up to version 4.

Motivation and Context

Dependency Cruiser does not currently support the latest version of Svelte, which is version 4. This update aims to address that. The syntax between Svelte 3 and 4 has not fundamentally changed, so only minimal modifications were needed to support version 4.

https://svelte.dev/docs/v4-migration-guide

How Has This Been Tested?

Screenshots

Types of changes

Checklist

socket-security[bot] commented 1 month ago

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/svelte@4.2.18 Transitive: unsafe +19 5.9 MB svelte-admin

🚮 Removed packages: npm/svelte@3.59.1

View full report↗︎

codeclimate[bot] commented 1 month ago

Code Climate has analyzed commit 759577da and detected 0 issues on this pull request.

View more on Code Climate.

sverweij commented 1 month ago

Hi @kubotak-is thanks for taking the time to research this and turning it in to a pull request! I was surprised adding support for svelte 4 looks so relatively straightforward.

Apparently an update to eslint and some caching crossed each other, resulting in a linting error that got nothing to do with your PR, sorry about that! I've added a commit (759577d2) to fix that.

kubotak-is commented 1 month ago

@sverweij Thank you for promptly merging the pull request.

sverweij commented 1 month ago

Hi @kubotak-is as we speak the code in this PR is being published to npm as version ~16.3.8~ 16.3.9. Thanks again!

(+ I've added you to the list of contributors in the package manifest)