Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 587614 - Sławek Lis (slis): multiple bad commit messages
Summary: Sławek Lis (slis): multiple bad commit messages
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Sławek Lis (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-06-30 20:01 UTC by Michał Górny
Modified: 2016-07-01 06:20 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2016-06-30 20:01:45 UTC
Sławek,

We noticed that you have performed multiple commits using unclear commit messages that do not follow our best practices. This is not the first time we are informed about this but I don't know if anyone has successfully contacted you before.

Please take a look at the 'git workflow' wiki page [1], in particular the part regarding commit messages [2]. Please make sure to use good commit messages suitable for git repository. This will really help other developers figuring out your commits later.

Please confirm that you have noticed this bug, and feel free to close it as soon as you acknowledge that you understand our request. Thanks in advance.

[1]:https://wiki.gentoo.org/wiki/Gentoo_git_workflow
[2]:https://wiki.gentoo.org/wiki/Gentoo_git_workflow#Commit_message_format
Comment 1 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2016-06-30 20:02:43 UTC
For reference, git shortlog provided by kensington:

Slawomir Lis (53):
      Added suricata ebuild (#437564)
      version bump (#567694)
      version bump
      version bump
      Fixed #569636
      version bump
      version bump, #569628
      fixed patch lost #569976, #569628
      version bump (#571352)
      Fixed tests (#571896)
      added assimp test patch
      django-classy-tags version bump
      django-sekizai version bump
      django-sekizai version bump
      djangocms-admin-style version bump
      django-cms version bump
      djangocms-text-ckeditor version bump
      Added detection USE flag, drop old
      fixing broken metadata.xml, sorry
      useflag to use libhtp instead of bundled one (#575144)
      drop system-htp use flag (#575144)
      version bump (#580824)
      added missing patch (#581256)
      version bump + #582598
      updated visit patchset (#582598)
      version bump
      version bump
      version bump
      version bump
      version bump
      version bump
      version bump
      version bump
      version bump
      version bump
      version bump
      version bump
      version bump
      version bump
      added YURL package
      added URLObject package
      fixed homepage
      added django-standard-form package
      added django-spurl package
      added django-sortedm2m package
      added aldryn-boilerplates package
      fixed metadata upstream
      added aldryn-common package
      added aldryn-search package
      updated Manifest
      version bump
      version bump
      version bump, metadata update

Sławomir Lis (27):
      Fixed #558470
      Fixed #558714
      Fixed deps (#558828)
      version bump (#555806)
      version bump (thanks to dxli)
      restoring stable version
      added fortran compiler check (#561136)
      Added django-classy-tags (#427728)
      Added package django-sekizai (#427734)
      Added django-treebeard package
      Added djangocms-admin-style package
      Added djangocms-file package
      Added djangocms-flash
      Added djangocms-inherit
      Added djangocms-link
      Added djangocms-picture
      added djangocms-snippet
      added djangocms-teaser
      added djangocms-text-ckeditor
      added djangocms-video
      Added django-cms (#427738)
      added django-mptt (#427730)
      added easy-thumbnails
      added django_polymorphic
      added django-filer
      added django-durationfield
      added dev-python/aldryn-bootstrap3
Comment 2 Sławek Lis (RETIRED) gentoo-dev 2016-07-01 06:11:36 UTC
Thanks for pointing out.