Skip to content

Conversation

@matt852
Copy link
Contributor

@matt852 matt852 commented Nov 26, 2025

Added some new variables from recent PR conversations to the data models. I'm looking to discuss these and have a conversation as to their validity.

@mjbear
Copy link
Collaborator

mjbear commented Nov 27, 2025

There are some more detailed descriptions I put in the data model on PR #2241

I have been thinking about opening an issue to discuss a few items rather than discuss it on 2241.

For example type checking (is it a list or a string) is simple and minimizes capture group sprawl. (I recognize the breaking behavior and there may likely be opposition. And certainly not something I'm immovable on.)

💡 Where capture group name normalization helps the community the most is when working with 2+ platforms for the same command(s)/protocol(s) in the same script, playbook, etc. (This is the reason I started some normalization.)

matt852 and others added 3 commits December 12, 2025 10:33
Co-authored-by: Michael Bear <38406045+mjbear@users.noreply.github.com>
Co-authored-by: Michael Bear <38406045+mjbear@users.noreply.github.com>
Co-authored-by: Michael Bear <38406045+mjbear@users.noreply.github.com>
@matt852 matt852 requested a review from mjbear December 12, 2025 15:34
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.

2 participants