Skip to content

recurring_lines.xml add tab as a valid character input#1597

Merged
bgruening merged 6 commits intobgruening:masterfrom
pavanvidem:pavanvidem-rl-patch
Mar 25, 2025
Merged

recurring_lines.xml add tab as a valid character input#1597
bgruening merged 6 commits intobgruening:masterfrom
pavanvidem:pavanvidem-rl-patch

Conversation

@pavanvidem
Copy link
Copy Markdown
Collaborator

@pavanvidem pavanvidem commented Mar 25, 2025

we are using this tool in IWC DE analysis workflow to create a header line that contains tabs. If we add tab here then we can avoid an extra step of sed and of course cleaner.

@mvdbeek
Copy link
Copy Markdown
Collaborator

mvdbeek commented Mar 25, 2025

Can you also add a test ?

pavanvidem and others added 2 commits March 25, 2025 12:41
Co-authored-by: Marius van den Beek <m.vandenbeek@gmail.com>
</xml>
<token name="@TOOL_VERSION@">9.5</token>
<token name="@VERSION_SUFFIX@">0</token>
<token name="@VERSION_SUFFIX@">1</token>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will bump the entire suite, and in return will bump all workflows using any of the tools in the suite. Is it not possible to override @VERSION_SUFFIX@ in the tool ?

Copy link
Copy Markdown
Collaborator Author

@pavanvidem pavanvidem Mar 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is possible but I think it is easy to maintain like this. If @bgruening prefers updating the version of this tool only, I will update it.

Copy link
Copy Markdown
Collaborator

@mvdbeek mvdbeek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@bgruening bgruening merged commit 7cdafed into bgruening:master Mar 25, 2025
14 checks passed
@pavanvidem pavanvidem deleted the pavanvidem-rl-patch branch March 25, 2025 15:51
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.

3 participants