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

Read multi slash branch names when deliver-work #256

Closed
wants to merge 1 commit into from
Closed

Conversation

extsoft
Copy link
Contributor

@extsoft extsoft commented Apr 15, 2020

git elegant deliver-work incorrectly reads the name of the upstream
branch if it contains at least one slash like feature/123. This is
because the used pattern matching logic doesn't expect remote branch
names like origin/feature/123.

In order to solve it, the new transformation plugin is introduced. It
provides functions that parse remote branch names correctly. Now, the
plugin handles branch translation (mapping) for deliver-work command
as well as for accept-work and obtain-work in order to prevent
possible errors in the future.

#255

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.

`git elegant deliver-work` incorrectly reads the name of the upstream
branch if it contains at least one slash like `feature/123`. This is
because the used pattern matching logic doesn't expect remote branch
names like `origin/feature/123`.

In order to solve it, the new `transformation` plugin is introduced. It
provides functions that parse remote branch names correctly. Now, the
plugin handles branch translation (mapping) for `deliver-work` command
as well as for `accept-work` and `obtain-work` in order to prevent
possible errors in the future.

#255
@extsoft extsoft closed this Apr 15, 2020
@extsoft extsoft deleted the bug/255 branch April 15, 2020 13:23
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