Skip to content

refactor: Enforce arrayRef regex & Handling and XML Spacing Rules #3900

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

refactor: Enforce arrayRef regex & Handling and XML Spacing Rules #3900

wants to merge 7 commits into from

Conversation

@arng40
Copy link
Contributor

@arng40 arng40 commented Nov 4, 2025 •
edited
Loading

To reference an array of groups aka groupNameRefArray, we ensure we have to write attribute="{*}" .
We cannot longer write for example : attribute="*"

Also allow to have spaces spaces before and after a value in an attribute :

someAttribute="x"
coordinates="    { 0.0, 1.0, 10.0, 100.0 }"

@arng40 arng40 self-assigned this Nov 4, 2025
@arng40 arng40 changed the title Enforce arrayRef regex Enforce arrayRef regex & loosens xml spacing rules Nov 4, 2025
@arng40 arng40 changed the title Enforce arrayRef regex & loosens xml spacing rules refactor: Enforce arrayRef regex & loosens xml spacing rules Nov 4, 2025
@arng40 arng40 changed the title refactor: Enforce arrayRef regex & loosens xml spacing rules refactor: Enforce arrayRef regex & Handling and XML Spacing Rules Nov 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rrsettgast rrsettgast Awaiting requested review from rrsettgast rrsettgast is a code owner

@paveltomin paveltomin Awaiting requested review from paveltomin paveltomin is a code owner

@CusiniM CusiniM Awaiting requested review from CusiniM CusiniM is a code owner

@herve-gross herve-gross Awaiting requested review from herve-gross herve-gross is a code owner

@castelletto1 castelletto1 Awaiting requested review from castelletto1 castelletto1 is a code owner

@jhuang2601 jhuang2601 Awaiting requested review from jhuang2601 jhuang2601 is a code owner

@dkachuma dkachuma Awaiting requested review from dkachuma dkachuma is a code owner

@OmarDuran OmarDuran Awaiting requested review from OmarDuran OmarDuran is a code owner

@corbett5 corbett5 Awaiting requested review from corbett5 corbett5 is a code owner

@wrtobin wrtobin Awaiting requested review from wrtobin wrtobin is a code owner

@MelReyCG MelReyCG Awaiting requested review from MelReyCG MelReyCG is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@arng40 arng40

Labels

flag: ready for review type: cleanup / refactor Non-functional change (NFC)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@arng40