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

Namespace support #60

Closed
wants to merge 75 commits into from
Closed

Namespace support #60

wants to merge 75 commits into from

Conversation

numen31337
Copy link
Owner

Closes #49

numen31337 and others added 24 commits October 25, 2020 18:46
* adding the failing test and shortening the extension name

* test fix

* adding a sophisticated test case

* fixing the generic annotation parsing

* comments, cleanup, and asserts

* extending _typeParametersNames function and removing _typeParametersAnnotation

* naming

* version bump

* minor correction
* basic implementation

* switch CI to dev

* minor corrections

* test fix

* changelog
* Updating dependencies

* typo fix
* readme update

* source_gen and build versions bump

* Update run_tests.yml
* added nullability for generic parameter constraints

* added nullability check for typing nullables
* analyzer version is set to 2.0.0

* ci minor corrections

* ci corrections

* a possible build fix

* minor ci correction

* changelog

* minor
* Switching to flutter_lints + null safety code cleanup

* CI corrections

* named constructor feature
* basic implementation

* minor

* prevent unnecessary copyWithNull methods generation

* naming corrections

* naming + ignore cast_nullable_to_non_nullable

* docs + test fixes

* docs

* docs

* release
* fix for copyWith null for a non-nullable value

* version bump

* minor ci correction

* minor ci correction
* Updating analyser version

* Release notes
* Lint correction

* tests correction
@numen31337
Copy link
Owner Author

Looks like this is a quite complex problem to solve, I looked at implementation of this feature in freezed library, it is able to handle some cases indeed, but not all of them e.g. final ns2.Future<ns1.ByteBuffer>? namespacedProperty2;. Perhaps I will have to postpone this problem for the future, as it requires time and much deeper understanding of the problem than I currently have.

@numen31337 numen31337 added help wanted Extra attention is needed enhancement New feature or request labels Aug 4, 2022
@numen31337 numen31337 marked this pull request as draft August 5, 2022 10:35
@numen31337 numen31337 closed this Jul 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to use CopyWith when models are imported with namespace
4 participants