-
Notifications
You must be signed in to change notification settings - Fork 42
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
Update repository documentation #41
Comments
Do we really need a changelog file? Pointing to the projects release page on github should do the trick |
A changelog is kinda required for breaking changes (at least).
|
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? |
@usox yes: send a PR for review :-) |
Well ok. Actually, I wanted to talk about dropping the phing targets before creating the pull request :-) |
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
(Dependencies have already been added to
composer.json
in Update readme + composer requirements; move license file and drop bui… #43, documentation needs to mention that the developer dependencies are added when runningcomposer install
).Other files
CONTRIBUTING.md
fileCOPYING
toLICENSE
- Update readme + composer requirements; move license file and drop bui… #43CHANGELOG
- Add a CHANGELOG #52Anybody else have opinions/thoughts/suggestions about this?
The text was updated successfully, but these errors were encountered: