shrinkwrap / descriptors

ShrinkWrap subproject for creating Archive Descriptors
Apache License 2.0
26 stars 30 forks source link

[SHRINKDESC-90] Extend test coverage of importers/exporters for SPI module #47

Closed bartoszmajsak closed 12 years ago

bartoszmajsak commented 12 years ago

Few more tests for Descriptors Importers/Exporters in the SPI module. Remaining parts which are not covered are related to some exceptions.

bartoszmajsak commented 12 years ago

Looks like I need to write pre-commit hook to append commit message with [BRANCH_NAME]. These typos are somehow embarrassing :)

ALRubinger commented 12 years ago

Hehe, don't worry about the commit messages. I rename or squash before upstream if there's any problems. But if you wanna fix 'em on your branch, just:

git rebase -i HEAD~N < Where N is the number of commits to rebase.

This will launch the editor, and change "pick" to "r", where upon save it'll give you the chance to rename.

Upstream: https://github.com/shrinkwrap/descriptors/commit/4fa7fa1233d300625078003719c7bf8cf879c8fb

bartoszmajsak commented 12 years ago

Thanks for the hint :)