Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added options to speficy defaults for poll frequence and vcs #181

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pfrandsen
Copy link

Added options to speficy defaults for ms-between-polls and vcs in config.json - if any of the attributes are not defined the hardcoded values in the code (30000, "git") will be used as in the current version of the application.

Example config.json

{
"dbpath" : "data",
"repo-defaults" : {
"ms-between-poll" : 30000,
"vcs" : "git"
},
"repos" : {
"Hound" : {
"url" : "https://www.github.com/etsy/Hound.git"
}
}
}

@pfrandsen
Copy link
Author

Added feature request #185

@bschlenk
Copy link
Contributor

This should be extended to the base-url and anchor as well. It would be useful to define these globally if you work at a company with a custom git browsing website.

Copy link
Contributor

@sahildua2305 sahildua2305 left a comment

Choose a reason for hiding this comment

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

Hi @pfrandsen would you still be interested in finishing off this patch? I left a minor improvement as a comment. Let's ship this!

@@ -33,6 +33,11 @@ type Repo struct {
EnablePushUpdates *bool `json:"enable-push-updates"`
}

type RepoDefaults struct {
Copy link
Contributor

Choose a reason for hiding this comment

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

The name of the struct should be singular. So, we can have RepoDefaultSetting or RepoDefault here.

Base automatically changed from master to main February 24, 2021 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants