Closed jsecrest closed 6 years ago
I've amended my commit to include the full stop as requested by li-boxuan
LGTM, a tiny change to go - could you add a full stop in your commit message.
This makes the dropdown list alphabetical
->This makes the dropdown list alphabetical.
I have added the trailing comma to the end of the list as requested. This pleases many contributors who like trailing commas and also prevents the unneeded line addition diff.
I also note that our commit documentation suggests that "If an issue is labelled with type/bug you should always use Fixes. For all other issues use Closes." As the label "type/bug" is on this issue I have changed "closes" to "fixes" in my commit.
Ack 500a0d7
@gitmate-bot ff
Hey! I'm GitMate.io! This pull request is being fastforwarded automatically. Please DO NOT push while fastforward is in progress or your changes would be lost permanently :warning:
Automated fastforward with GitMate.io was successful! :tada:
This makes the dropdown list alphabetical
Closes https://github.com/coala/landing-frontend/issues/276
For short term contributors: we understand that getting your commits well defined like we require is a hard task and takes some learning. If you look to help without wanting to contribute long term there's no need for you to learn this. Just drop us a message and we'll take care of brushing up your stuff for merge!
Checklist
After you submit your pull request, DO NOT click the 'Update Branch' button. When asked for a rebase, consult coala.io/rebase instead.
Please consider helping us by reviewing other peoples pull requests as well:
cobot mark wip <URL>
to get it out of the review queue.The more you review, the more your score will grow at coala.io and we will review your PRs faster!