Skip to content

feat(nagelfar): Support file name prefix in regex#5141

Open
MayeulC wants to merge 2 commits into
dense-analysis:masterfrom
MayeulC:nagelfar-multifile-fix
Open

feat(nagelfar): Support file name prefix in regex#5141
MayeulC wants to merge 2 commits into
dense-analysis:masterfrom
MayeulC:nagelfar-multifile-fix

Conversation

@MayeulC

@MayeulC MayeulC commented Jun 12, 2026

Copy link
Copy Markdown

This allows adding a header file containing custom definitions to tcl_nagelfar_otions.

Nagelfar can generate such headers with:
nagelfar -header file_name

Note that the file name displayed is the path that is passed as a parameter to nagelfar.

I tested the change, it works both with an extra header and without.

I have added a test, but:

  • I did not run the test (hoping it passes CI)
  • I am unsure if I should make a bare call to file or use ale#test#SetFilename('def.tcl') + ale#test#GetFilename('def.tcl')

Edit: I only have access to the web GitHub UI right now, and I committed the test to the wrong branch. Fixed, but this cancelled the workflow.

This allows adding a header file containing custom definitions to tcl_nagelfar_otions.

Nagelfar can generate such headers with:
nagelfar -header file_name <input tcl files>
@MayeulC MayeulC requested a review from w0rp as a code owner June 12, 2026 13:31
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.

1 participant