You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@age.apache.org by "jrgemignani (via GitHub)" <gi...@apache.org> on 2023/03/17 16:48:20 UTC
[GitHub] [age] jrgemignani opened a new pull request, #751: Fix property constraints against resolved variables (#724)
jrgemignani opened a new pull request, #751:
URL: https://github.com/apache/age/pull/751
Fixed an issue where an already resolved variable, when used for
a property constraint, errored out. See issue #724 for more details.
This is the second part of the fix for the match property constraint
bug regarding variable reuse. The prior fix addressed clause-to-clause,
this fix addresses within a clause.
Adjusted and added additional regression tests.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@age.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [age] dehowef merged pull request #751: Fix property constraints against resolved variables (#724)
Posted by "dehowef (via GitHub)" <gi...@apache.org>.
dehowef merged PR #751:
URL: https://github.com/apache/age/pull/751
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@age.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [age] dehowef commented on pull request #751: Fix property constraints against resolved variables (#724)
Posted by "dehowef (via GitHub)" <gi...@apache.org>.
dehowef commented on PR #751:
URL: https://github.com/apache/age/pull/751#issuecomment-1479852199
I have reviewed these changes with John and have tested them and they look good and align with the expected behavior that AGE should have
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@age.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [age] jrgemignani commented on pull request #751: Fix property constraints against resolved variables (#724)
Posted by "jrgemignani (via GitHub)" <gi...@apache.org>.
jrgemignani commented on PR #751:
URL: https://github.com/apache/age/pull/751#issuecomment-1479924651
@dehowef Updated PR to include label checking and additional regression tests.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@age.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [age] dehowef commented on pull request #751: Fix property constraints against resolved variables (#724)
Posted by "dehowef (via GitHub)" <gi...@apache.org>.
dehowef commented on PR #751:
URL: https://github.com/apache/age/pull/751#issuecomment-1479950315
Looked over the additional regression tests, and they look good to me.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@age.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org