Skip to content
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.

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

Adding SyntaxContext #2245

Draft
wants to merge 11 commits into
base: develop/3.0
Choose a base branch
from
Draft

Conversation

curin
Copy link
Contributor

@curin curin commented Jul 11, 2024

Summary of the PR

Adds the SyntaxContext object to better interact with the whole syntax tree. Reworks the mods system to use this instead of the GeneratedSyntax object, which is instead used as part of the SyntaxContext construction.

Initial commit
Unified TypeContextContainer code
Added Parameter tracking for methods
-added some BaseType tracking
-added member tracking in Enums for easier combining
-added support for generic types
-finished Enum rewrite
-working on other types rewrite (class, struct, record, etc.)
-added new CSharpSyntaxRewriter to hold extra metadata
-Added Query functions
-Finished Type Rewrites
-Stubbed in both Namespace and CompilationUnit Rewrites
-Moved IBaseTypeContext out to its own file
-also fixed some bugs
Also added some missing code for rewrite
-Adding support for Delegates, Indexers, and Events
-Delegates is incomplete
-added some type cleanup as well
Also added to the interfaces and fixed some potential issues related to editing a list within a foreach loop
@Perksey
Copy link
Member

Perksey commented Nov 7, 2024

Is this dead now we're delegating to Roslyn?

@Perksey Perksey closed this Nov 7, 2024
@Perksey Perksey reopened this Nov 7, 2024
@Perksey
Copy link
Member

Perksey commented Nov 7, 2024

I meant to ask a question not answer it lmao

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

2 participants