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.

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

Fix enum and type resolution #309

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Fix enum and type resolution #309

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
6ce37a4
Rename auxiliary functions that implements resolving type and enum names
Mingun Sep 26, 2024
9cc21cf
Wrap type name in '' because it is not always clear that part of a me…
Mingun Sep 26, 2024
04ba421
Add tests for type name resolution
Mingun Sep 26, 2024
4ec107f
Fix type resolution in expressions
Mingun Sep 26, 2024
ce3875e
Add tests for enum resolution in expressions
Mingun Sep 26, 2024
8033407
Fix enum resolution in expressions
Mingun Sep 26, 2024
374dc8b
Remove now used only in one place `ClassTypeProvider.resolveClassSpec`
Mingun Sep 26, 2024
64d770b
Make all internal methods of ResolveTypes pass private
Mingun Sep 26, 2024
9cac8d2
Fix enum resolution in `enum` keys
Mingun Sep 26, 2024
7697cac
Fix type resolution in `type` keys
Mingun Sep 26, 2024
a4706e7
Remove unnecessary variable and matching
Mingun Sep 26, 2024
3e68dcc
Inline real type resolution method
Mingun Sep 26, 2024
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
Loading
Loading