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

Validate templated field assigned also in super constructor #37216

Open
1 of 2 tasks
romsharon98 opened this issue Feb 7, 2024 · 1 comment
Open
1 of 2 tasks

Validate templated field assigned also in super constructor #37216

romsharon98 opened this issue Feb 7, 2024 · 1 comment
Labels

Comments

@romsharon98
Copy link
Contributor

romsharon98 commented Feb 7, 2024

Description

the issue is follow up of this pr: #33786 that add the abilty to checks for the templated field "invalid logic" within the constructor of providers' operators.
the issue comes from

TODO: Enhance this function to work with nested inheritance trees through dynamic imports.

and it will add the ability to validate assignment of templated field in super constructor.

Use case/motivation

this issue will close pr's like this #36789 and will prevent similar errors like from this pr #35278.
and probably will solve errors from this list #36484

Related issues

#36484
#29069

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@romsharon98 romsharon98 added kind:feature Feature Requests needs-triage label for new issues that we didn't triage yet labels Feb 7, 2024
Copy link

boring-cyborg bot commented Feb 7, 2024

Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.

@Taragolis Taragolis added area:dev-tools and removed needs-triage label for new issues that we didn't triage yet labels Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants