Skip to content
This repository was archived by the owner on Mar 5, 2025. It is now read-only.

Split deploy git variables #1341

Closed
wants to merge 2 commits into from

Conversation

marvil07
Copy link
Contributor

@marvil07 marvil07 commented Apr 7, 2017

Fixes #1340.

Changes proposed:

  • Split git config overrides on deploy git directory, so both of them are not required.
  • Documents git.user.name and git.user.email blt options.

@marvil07
Copy link
Contributor Author

marvil07 commented Apr 7, 2017

This expands #1323.

<isset property="git.user.name"/>
<then>
<exec dir="${deploy.dir}" command="git config --local --add user.name ${git.user.name}" logoutput="true" checkreturn="true" level="${blt.exec_level}" passthru="true"/>
</if>
Copy link
Contributor

Choose a reason for hiding this comment

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

XML tags are not properly closed here.

@grasmash
Copy link
Contributor

Closing due to syntax error. Waiting on response in #1340.

@grasmash grasmash closed this Apr 14, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants