Skip to content

Feature/config manager pull custom nodes#98

Open
dallinjsevy wants to merge 1 commit into
mainfrom
feature/config-manager-pull-custom-nodes
Open

Feature/config manager pull custom nodes#98
dallinjsevy wants to merge 1 commit into
mainfrom
feature/config-manager-pull-custom-nodes

Conversation

@dallinjsevy

Copy link
Copy Markdown

No description provided.

@dallinjsevy dallinjsevy force-pushed the feature/config-manager-pull-custom-nodes branch from f0a1cde to 5c16f81 Compare May 26, 2026 17:10
Comment thread src/cli/config-manager/config-manager-push/config-manager-push-custom-nodes.ts Outdated
Comment thread src/cli/config-manager/config-manager-push/config-manager-push-custom-nodes.ts Outdated

export default function setup() {
const program = new FrodoCommand(
'frodo config-manager pull custom-nodes',

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Should say push instead of pull

Comment thread src/cli/config-manager/config-manager-push/config-manager-push-custom-nodes.ts Outdated
Comment thread src/cli/config-manager/config-manager-push/config-manager-push-custom-nodes.ts Outdated
Comment thread src/configManagerOps/FrConfigCustomNodesOps.ts Outdated
Comment thread test/e2e/mocks/github_584874290/recording.har
Comment thread test/e2e/mocks/npmjs_1455397529/recording.har
Comment on lines +70 to +71
const { stdout } = await exec(CMD, forgeopsEnv);
expect(removeAnsiEscapeCodes(stdout)).toMatchSnapshot();

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggestion: I would replace these lines in your test code to be:

await testSuccess(CMD, forgeopsENV);

This helper I created just helps simplify the testing code some more

Comment thread src/configManagerOps/FrConfigCustomNodesOps.ts
@dallinjsevy dallinjsevy force-pushed the feature/config-manager-pull-custom-nodes branch from 5c16f81 to 7f2fc64 Compare July 16, 2026 15:09
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