Skip to content

Fix '.search also' with /handler config=full or existing - #1144

Open
fritzw wants to merge 2 commits into
pgf-tikz:masterfrom
fritzw:fix-search-also
Open

Fix '.search also' with /handler config=full or existing#1144
fritzw wants to merge 2 commits into
pgf-tikz:masterfrom
fritzw:fix-search-also

Conversation

@fritzw

@fritzw fritzw commented Mar 9, 2022

Copy link
Copy Markdown

Make .expand* keys use relative paths if the original path was relative.

Signed-off-by: Fritz Webering fritz@webering.eu

Motivation for this change

Fixes #1143

Checklist

Please signoff your commits to explicitly state your agreement to the Developer Certificate of Origin. If that is not possible you may check the boxes below instead:

@fritzw
fritzw force-pushed the fix-search-also branch from 2265a5e to bb1436b Compare March 9, 2022 23:23
fritzw and others added 2 commits March 10, 2022 00:24
Make .expand* keys use relative paths if the original path was relative.

Signed-off-by: Fritz Webering <fritz@webering.eu>
Add missing 'only existing' exceptions.
Expand the example explaining the interaction
between '.search also' and '/handler config'.

Signed-off-by: Fritz Webering <fritz@webering.eu>
@fritzw
fritzw force-pushed the fix-search-also branch from bb1436b to f857fc8 Compare March 9, 2022 23:25
@hmenke

hmenke commented Jun 16, 2026

Copy link
Copy Markdown
Member

On 2026-06-15 a more comprehensive testsuite was introduced for PGF/TikZ. This tests all new contributions against the examples extracted from the manual and checks whether anything has changed to avoid regressions. Since this pull request was opened before that, these tests were not run automatically. I have run l3build check on the result of the merge of this pull request branch into master. The result can be found below.

Tip

Your pull request might have failed some tests. This does not mean that your contribution will be rejected. In fact, depending on the changes you propose, it might even be expected that some tests fail, however, this needs to be evaluated carefully.

All tests passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Handler config 'full or existing' can set non-existent keys when '.search also' is in effect.

2 participants