Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Refactor #125

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

Merged
woile merged 11 commits into commitizen-tools:master from Lee-W:refactor
Jan 21, 2020
Merged

Refactor #125

woile merged 11 commits into commitizen-tools:master from Lee-W:refactor
Jan 21, 2020

Conversation

Copy link
Member

@Lee-W Lee-W commented Jan 21, 2020
edited
Loading

  • modify the way commitizen.cmd interact with shell
    • the original design fails to handle git tag with long arguments
  • use GitObject (GitTag, GitCommit) classes to store git information
    • add get git tags, commits functions
  • rename variables
  • add test cases and reorganize existing test cases

I start working on changelog (#53) on my changelog branch and have a prototype of it.
However, the pull request will be too huge to review if I send it after I finish all the functionality.
Thus, I separate the changelog related commits and other refactor commits which are changed as a preparation for the new functionality.
This PR is for the refactor ones.

Lee-W added 11 commits January 21, 2020 13:14
In the original design, command to long will not be run.
I use the code how invoke runs sunbprocess.
https://github.com/pyinvoke/invoke/blob/master/invoke/runners.py#L1271 
... into conftest
they're commonly used fixture among all tests
@Lee-W Lee-W requested a review from woile January 21, 2020 11:29
@woile woile merged commit cbc145e into commitizen-tools:master Jan 21, 2020
Copy link
Member

woile commented Jan 21, 2020

Thanks Lee!

Copy link
Member Author

Lee-W commented Jan 21, 2020

Thank you for the prompt review 🚀

@Lee-W Lee-W deleted the refactor branch February 20, 2020 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@woile woile Awaiting requested review from woile woile is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants

AltStyle によって変換されたページ (->オリジナル) /