Skip to content

Support skipping configs during configuration and verification#74

Merged
texasaggie97 merged 3 commits intoni:masterfrom
texasaggie97:dev/texasaggie97/allow-disable-of-each-config
Sep 24, 2025
Merged

Support skipping configs during configuration and verification#74
texasaggie97 merged 3 commits intoni:masterfrom
texasaggie97:dev/texasaggie97/allow-disable-of-each-config

Conversation

@texasaggie97
Copy link
Copy Markdown
Collaborator

@texasaggie97 texasaggie97 commented Sep 19, 2025

Summary of Changes

There have been requests to allow skipping the configuration steps since they are difficult to undo. If the configuration is skipped, we need to also allow skipping the verification of that config.

Justification

AB#3292328

Testing

PR Tests

Procedure

  • This PR: changes user-visible behavior, fixes a bug, or impacts the project's security profile; and so it includes a CHANGELOG note.
  • I certify that the contents of this pull request complies with the Developer Certificate of Origin.

@texasaggie97 texasaggie97 force-pushed the dev/texasaggie97/allow-disable-of-each-config branch 2 times, most recently from 12f3efb to a435898 Compare September 22, 2025 17:45
@texasaggie97 texasaggie97 force-pushed the dev/texasaggie97/allow-disable-of-each-config branch from a435898 to aee3acf Compare September 22, 2025 21:05
@texasaggie97 texasaggie97 force-pushed the dev/texasaggie97/allow-disable-of-each-config branch from aee3acf to 9dee3f3 Compare September 23, 2025 19:09
@texasaggie97 texasaggie97 dismissed amstewart’s stale review September 23, 2025 19:29

Comments addressed

structure and enable module control via config file

Signed-off-by: Mark Silva <mark.silva@emerson.com>
Signed-off-by: Mark Silva <mark.silva@emerson.com>
@amstewart
Copy link
Copy Markdown
Collaborator

@texasaggie97 Could you resolve your conflicts with #73 and merge?

Signed-off-by: Mark Silva <mark.silva@emerson.com>
@texasaggie97 texasaggie97 force-pushed the dev/texasaggie97/allow-disable-of-each-config branch from a65fe9f to 9e6fd1f Compare September 24, 2025 17:58
@texasaggie97 texasaggie97 merged commit 9f01d8b into ni:master Sep 24, 2025
5 checks passed
@texasaggie97 texasaggie97 deleted the dev/texasaggie97/allow-disable-of-each-config branch September 24, 2025 18:18
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.

4 participants