PHPTAL icon indicating copy to clipboard operation
PHPTAL copied to clipboard

Update repository documentation

Open Potherca opened this issue 10 years ago • 5 comments

Various parts of the README file need updating or changing to be more up-to-date. Also, some additions seem in order.

My suggestions would be to:

Readme File

  • [x] Update the PHPUnit version - #43
  • [x] Add install instructions for Composer - #43
  • [x] Remove the PEAR section in favour of using Composer/Packagist - #43
  • [x] Point the package link to Github releases rather than to http://phptal.org - #43
  • [x] Point the development link to Github rather than https://svn.motion-twin.com/ - #43
  • [x] Add a license badge - #43
  • [x] Add a Packagist badge - #43
  • [x] Add Scrutinizer badge - #43
  • [x] Update the location of the Travis build: Build Status - #43
  • [ ] Update the development section to use Composer to install development dependencies
    (Dependencies have already been added to composer.json in #43, documentation needs to mention that the developer dependencies are added when running composer install).
  • [ ] Add Credits to honor the original authors and to add the maintainers (to avoid confusion about who is responsible for what)

Other files

  • [ ] Add a CONTRIBUTING.md file
  • [x] Rename COPYING to LICENSE - #43
  • [ ] Add a CHANGELOG - #52

Anybody else have opinions/thoughts/suggestions about this?

Potherca avatar Aug 31 '15 09:08 Potherca

Add a CHANGELOG

Do we really need a changelog file? Pointing to the projects release page on github should do the trick

usox avatar Sep 01 '15 09:09 usox

A changelog is kinda required for breaking changes (at least). On Sep 1, 2015 10:42 AM, "usox" [email protected] wrote:

Add a CHANGELOG

Do we really need a changelog file? Pointing to the projects release page on github should do the trick

— Reply to this email directly or view it on GitHub https://github.com/phptal/PHPTAL/issues/41#issuecomment-136651425.

Ocramius avatar Sep 01 '15 09:09 Ocramius

I updated the readme and the composer.json in our fork some time ago so I tried to backport some of the changes including some of the proposed suggestions. Any thoughts? https://github.com/usox/PHPTAL/tree/WIP/readme_and_composer_update

usox avatar Sep 01 '15 10:09 usox

@usox yes: send a PR for review :-)

Ocramius avatar Sep 01 '15 10:09 Ocramius

Well ok. Actually, I wanted to talk about dropping the phing targets before creating the pull request :-)

usox avatar Sep 01 '15 11:09 usox