Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Support alternate config dir for GitGitGadget#165

Merged
dscho merged 1 commit intogitgitgadget:mastergitgitgadget/gitgitgadget:masterfrom
webstech:alternate-configwebstech/gitgitgadget:alternate-configCopy head branch name to clipboard
Dec 9, 2019
Merged

Support alternate config dir for GitGitGadget#165
dscho merged 1 commit intogitgitgadget:mastergitgitgadget/gitgitgadget:masterfrom
webstech:alternate-configwebstech/gitgitgadget:alternate-configCopy head branch name to clipboard

Conversation

@webstech
Copy link
Contributor

@webstech webstech commented Dec 9, 2019

Positioning change for tests which need to provide a separate
config to GitGitGadget.get(). Currently, the current directory
is passed. This will remain the default but user testing should
not need to update the local repo config. This change will allow
tests to provide an alternate location of the config which
GitGitGadget uses to obtain SMTP information.

Issue #140 and PR #156 will be able to use this change. A work in progress at webstech/ci-helper-test is a sample use of this change.

Positioning change for tests which need to provide a separate
config to GitGitGadget.get().  Currently, the current directory
is passed.  This will remain the default but user testing should
not need to update the local repo config.  This change will allow
tests to provide an alternate location of the config which
GitGitGadget uses to obtain SMTP information.

Signed-off-by: Chris. Webster <chris@webstech.net>
@webstech webstech requested a review from dscho December 9, 2019 06:47
@dscho dscho merged commit 91324cb into gitgitgadget:master Dec 9, 2019
@dscho
Copy link
Member

dscho commented Dec 9, 2019

Thank you!

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.