Skip to content

clijs: implement config commands#879

Merged
Rolaman merged 9 commits intomainfrom
cli-config
May 8, 2025
Merged

clijs: implement config commands#879
Rolaman merged 9 commits intomainfrom
cli-config

Conversation

@Rolaman
Copy link
Copy Markdown
Contributor

@Rolaman Rolaman commented May 5, 2025

Short Summary

Implement config commands

What Changes Were Made

  • config init
  • config get
  • config set
  • config show

Fixed #454

Checklist

  • [v] I have read and followed the Contributing Guide
  • [v] I have tested the changes locally
  • [v] I have added relevant tests (if applicable)
  • [v] I have updated documentation/comments (if applicable)

@Rolaman Rolaman self-assigned this May 5, 2025
@Rolaman Rolaman marked this pull request as ready for review May 5, 2025 07:49
@Rolaman Rolaman requested review from Gezort, olegrok and ukorvl May 5, 2025 07:51
expect(result).to.contains(testValue);
});

CliTest("tests config:init command", async ({ runCommand, cfgPath }) => {
Copy link
Copy Markdown
Contributor

@Un1oR Un1oR May 5, 2025

Choose a reason for hiding this comment

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

Also of interest is the -c test.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor

@Un1oR Un1oR May 7, 2025

Choose a reason for hiding this comment

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

Still don't see that test

@Rolaman Rolaman added this pull request to the merge queue May 8, 2025
Merged via the queue into main with commit f00438c May 8, 2025
16 checks passed
@Rolaman Rolaman deleted the cli-config branch May 8, 2025 10:53
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.

[cli.js] Support config.ini

3 participants