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

Add a comment to the manual's section about MatcherRules regarding unqualified / qualified identifier matching #5767

Closed
wants to merge 1 commit into from

Conversation

deinspanjer
Copy link

Add note about the fact the expression is tested against the unqualified name first then the qualified name.
Might help prevent confusion similar to this thread:
https://groups.google.com/forum/#!topic/jooq-user/XwRzNf5Gtjs

Add note about the fact the expression is tested against the unqualified name first then the qualified name.
Might help prevent confusion similar to this thread:
https://groups.google.com/forum/#!topic/jooq-user/XwRzNf5Gtjs
@lukaseder
Copy link
Member

Thank you very much for your suggestion. I will implement the improvement slightly differently:

@lukaseder lukaseder changed the title Add a note to field MatcherRule example Add a comment to the manual's section about MatcherRules regarding unqualified / qualified identifier matching Jan 5, 2017
lukaseder added a commit that referenced this pull request Jan 5, 2017
…rding unqualified / qualified identifier matching
@lukaseder lukaseder closed this Jan 5, 2017
@deinspanjer
Copy link
Author

That is great, I'm glad that this text could be of some use. I just did not want to make a bunch of suggestions on things to do and not offer any assistance.

@lukaseder
Copy link
Member

Hey, sure. Your assistance is greatly appreciated. Looking forward to more of it! :)

@deinspanjer deinspanjer deleted the patch-1 branch January 5, 2017 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants