Skip to content

Rename configTagList to groupTagList#186

Merged
pr0uxx merged 2 commits into
mainfrom
SHIELD-Architecture-Spec-BugFix
Jul 1, 2026
Merged

Rename configTagList to groupTagList#186
pr0uxx merged 2 commits into
mainfrom
SHIELD-Architecture-Spec-BugFix

Conversation

@Cevan00

Copy link
Copy Markdown
Contributor

This pull request makes a small update to the specs/SHIELD.json file, renaming the configTagList property to groupTagList to better reflect its purpose.

Update property name in SHIELD.json schema to better reflect its purpose as a group tag list rather than configuration tag list.
Copilot AI review requested due to automatic review settings June 30, 2026 22:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the SHIELD OpenAPI/JSON spec to rename a schema property from configTagList to groupTagList in order to better reflect its intended meaning within the Deploy architecture metadata model.

Changes:

  • Renamed configTagList to groupTagList in specs/SHIELD.json (within the Deploy.ArchitectureData schema).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread specs/SHIELD.json
Updates the SHIELD OpenAPI schema to require `groupTagList` instead of `configTagList`, aligning the required fields with the current property naming.
@pr0uxx
pr0uxx self-requested a review July 1, 2026 13:27
@pr0uxx
pr0uxx merged commit 36e7a11 into main Jul 1, 2026
8 checks passed
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