Skip to content

Feat roslyn analyzer support#168

Merged
handzlikchris merged 1 commit intohandzlikchris:masterfrom
labbbirder:feat-roslyn-analyzer-support
Apr 23, 2025
Merged

Feat roslyn analyzer support#168
handzlikchris merged 1 commit intohandzlikchris:masterfrom
labbbirder:feat-roslyn-analyzer-support

Conversation

@labbbirder
Copy link
Contributor

Add supports to roslyn analyer(source generator, more exactly).

@labbbirder labbbirder force-pushed the feat-roslyn-analyzer-support branch from c705b6e to 4075f1a Compare April 23, 2025 06:35
@handzlikchris
Copy link
Owner

Looks good, thanks for contribution!

@handzlikchris
Copy link
Owner

the only thing would be performance of AssetDatabase.FindAssets("t: " + nameof(DefaultAsset)) - especially on big project. I'll have a peek at that later

@handzlikchris handzlikchris merged commit 3d93661 into handzlikchris:master Apr 23, 2025
@labbbirder
Copy link
Contributor Author

the only thing would be performance of AssetDatabase.FindAssets("t: " + nameof(DefaultAsset)) - especially on big project. I'll have a peek at that later

Yes, you are right.

Maybe reading xml-document from *.csproj could be better. These are files auto generated by builtin unity package Visual Studio Code Editor or Visual Studio Editor.

@labbbirder labbbirder deleted the feat-roslyn-analyzer-support branch April 24, 2025 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants