Skip to content
This repository has been archived by the owner on May 3, 2020. It is now read-only.

Ignore merge commits #60

Closed
1 task
jirikuncar opened this issue Mar 9, 2015 · 0 comments · Fixed by #77
Closed
1 task

Ignore merge commits #60

jirikuncar opened this issue Mar 9, 2015 · 0 comments · Fixed by #77
Assignees
@tiborsimko
Labels
Lane: help wanted Type: enhancement
Milestone
v0.2.0

Comments

@jirikuncar
Copy link
Contributor

  • add option to skip merge commits kwalitee check
tiborsimko added a commit to tiborsimko/kwalitee that referenced this issue Aug 19, 2015
@tiborsimko
cli: new CLI option to skip checking merge commits
6a82580 
* NEW Adds a new CLI option `kwalitee check -s` to skip checking of
  merge commits.  (closes inveniosoftware-attic#60)

* Improves PEP-8 compliance of the file.

Signed-off-by: Tibor Simko <tibor.simko@cern.ch>
@tiborsimko tiborsimko self-assigned this Aug 19, 2015
@tiborsimko tiborsimko added this to the v0.2.0 milestone Aug 19, 2015
@tiborsimko tiborsimko mentioned this issue Aug 19, 2015
Merged
tiborsimko added a commit to tiborsimko/kwalitee that referenced this issue Aug 19, 2015
@tiborsimko
cli: new check option to skip merge commits
2ac284d 
* NEW Adds a new CLI option `-s, --skip-merge-commits` to both
  `kwalitee check` commands.  (closes inveniosoftware-attic#60)

* Adds test cases for both `pygit2` and `GitPython` to read git
  repository logs and detect merge commits.

* Improves PEP-8 compliance.

Signed-off-by: Tibor Simko <tibor.simko@cern.ch>
Reviewed-by: Jiri Kuncar <jiri.kuncar@cern.ch>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@tiborsimko tiborsimko

Labels
Lane: help wanted Type: enhancement
Projects
None yet
Milestone
v0.2.0
Development

Successfully merging a pull request may close this issue.

cli: new CLI option to skip checking merge commits tiborsimko/kwalitee
2 participants
@jirikuncar @tiborsimko