Skip to content

Comments

feat: added custom config support#16

Merged
Gustash merged 1 commit intomainfrom
feat/custom-config-support
Dec 12, 2025
Merged

feat: added custom config support#16
Gustash merged 1 commit intomainfrom
feat/custom-config-support

Conversation

@Gustash
Copy link
Owner

@Gustash Gustash commented Dec 12, 2025

Closes #13

Copilot AI review requested due to automatic review settings December 12, 2025 04:00
Copy link
Contributor

Copilot AI left a comment

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 adds support for custom configuration directory paths through a new --config-path command-line flag. This allows users to specify an alternative location for the application's configuration files instead of using the default directory.

Key Changes:

  • Added --config-path flag to allow users to specify a custom configuration directory
  • Renamed testConfigDir to overrideConfigDir to better reflect its dual purpose (testing and custom config)
  • Updated all test files to use the renamed variable

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
main.go Added configPath flag, import for auth package, and logic to set custom config directory using absolute paths
go.mod Moved github.com/google/shlex from indirect to direct dependency
auth/config.go Renamed testConfigDir to overrideConfigDir and added OverrideConfigDir function for public use
auth/session_test.go Updated variable references from testConfigDir to overrideConfigDir
auth/userinfo_test.go Updated variable references from testConfigDir to overrideConfigDir

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

@Gustash Gustash merged commit 02e4b4f into main Dec 12, 2025
9 checks passed
@Gustash Gustash deleted the feat/custom-config-support branch December 12, 2025 04:05
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.

Custom config path

1 participant