Skip to content

add git_lock_timeout in values.yml and make the builder-deployment.ya…#61

Closed
robinmonjo wants to merge 1 commit into
teamhephy:masterfrom
robinmonjo:make-git-lock-timeout-configurable
Closed

add git_lock_timeout in values.yml and make the builder-deployment.ya…#61
robinmonjo wants to merge 1 commit into
teamhephy:masterfrom
robinmonjo:make-git-lock-timeout-configurable

Conversation

@robinmonjo
Copy link
Copy Markdown
Contributor

@robinmonjo robinmonjo commented Jan 30, 2020

…ml use it

If your change requires any additions or changes to the documentation or to the end-to-end test suite, please submit them as 1 or more pull requests against that repo and refer to them here.

@Cryptophobia Cryptophobia self-requested a review January 30, 2020 17:15
Copy link
Copy Markdown
Member

@Cryptophobia Cryptophobia left a comment

Choose a reason for hiding this comment

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

Thanks @robinmonjo ! This is a great change. Thank you for opening the PR.

Just one small change request, could you reformat the commit msg so it follows this style:
https://docs.teamhephy.com/contributing/submitting-a-pull-request/#commit-style

Something like this would work out perfectly for our release tool:

"feat(values): add git_lock_timeout in values.yml for builder"

@Cryptophobia Cryptophobia requested a review from kingdonb January 30, 2020 18:18
kingdonb pushed a commit to kingdonb/builder that referenced this pull request Feb 1, 2020
Add git_lock_timeout in values.yml and make the builder-deployment.yaml
use it.

closes teamhephy#61
@kingdonb
Copy link
Copy Markdown
Member

kingdonb commented Feb 1, 2020

I made a PR with the same content, but conformant commit message (see #62) - this closes #61

kingdonb pushed a commit to kingdonb/builder that referenced this pull request Feb 1, 2020
Add git_lock_timeout in values.yml and make the builder-deployment.yaml
use it.

closes teamhephy#61
@Cryptophobia
Copy link
Copy Markdown
Member

Merged!

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.

3 participants