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

Improve configuration experience #218

Merged
merged 3 commits into from Nov 3, 2019
Merged

Improve configuration experience #218

merged 3 commits into from Nov 3, 2019

Conversation

extsoft
Copy link
Contributor

@extsoft extsoft commented Nov 3, 2019

The proposed changes appertain to #177.

The contribution:

  • updates all affected documentation
  • provides commits messages which comply with the CONTRIBUTING.md > Committing the changes rules
  • updates the completion scripts if requires
  • complies with all requirements from README.md > Hands-on development notes

@bees-hive/elegant-git-maintainers, please review.

First of all, the configuration logic is described in the documentation.
This gives a user more details on what will be configured.

Next, the configuration logic is moved to a separate plugin. This allows
to reuse it in the future. All tests were updated also.

#177
The command allows applying the required configuration to a Git
installation. Since there are some interceptionsbetween the
`acquire-git` and `acquire-repository`, there are some changes in the
overall configuration workflow. These changes are desribed on the
documentation.

Also, to prevent unexpected tests failures, the `~/.gitconfig` is
removed after each test where a real Git repository is used.

#177
The configuration will start from `acquire-git` command and guide a user
through the process interactively. If standards are configured, they
will be printed to CLI instead of reconfiguring.

During the installation, `acquire-git` will be executed instead of
providing the instructions. This allows updating settings for the new
versions automatically.

Now, the `read` addon supports partial mocking of the answers correctly.
This works because a replay variable is always set with either provided
value or empty value.

#177
@extsoft extsoft merged commit 0d70a6e into master Nov 3, 2019
@extsoft extsoft deleted the 177 branch November 3, 2019 12:08
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.

None yet

1 participant