Skip to content
This repository has been archived by the owner on Sep 27, 2023. It is now read-only.

Fix template replacement order #69

Open
Adrodoc opened this issue Apr 25, 2022 · 0 comments
Open

Fix template replacement order #69

Adrodoc opened this issue Apr 25, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@Adrodoc
Copy link
Member

Adrodoc commented Apr 25, 2022

Currently some template replacements are only done incidentally. For instance # -commands- in the validate template gets replaced with commands containing -ns- which is then later replaced with the actual namespace. This only works because we use a BTreeMap and the UTF-8 value of # is smaller than the value of -, so # -commands- is replaced first. This replacement should not rely on such implicit ordering.

@Adrodoc Adrodoc added the enhancement New feature or request label Apr 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant