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

fix(input-migration): enhance regexp to consider whitespaces when matching against temp… #409

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ilirbeqirii
Copy link
Contributor

…late

Closes #397 (I also tried to work on the case mentioned on the issue, but am not sure I did it right. It would be helpful if we could have a more complete sample of non-working migration for the opened issue.)

When testing for matches in the template, please make sure we consider when the template code is not properly formatted and the interpolation expression has spaces on both sides.

@ilirbeqirii ilirbeqirii force-pushed the fix/signal-input-migration-chaining-operator-template branch from 6b2ddef to e3ef9fe Compare June 7, 2024 21:49
@nartc nartc requested a review from eneajaho June 10, 2024 04:05
@nartc nartc assigned nartc and eneajaho and unassigned nartc Jun 10, 2024
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.

Template migration fails when using optional chaining operator in templates
3 participants