Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Improve contributing section #20

Merged
merged 1 commit into from
Jun 13, 2016

Conversation

abelsoares
Copy link
Contributor

@abelsoares abelsoares commented Jun 13, 2016

This PR improves the README.md contributing section.

Closes #19.


### Before you start

- `bookshelf-json-columns` supports PostgreSql and sqlite3.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use bold.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

... PostgreSQL and SQLite3.

@abelsoares abelsoares force-pushed the enhancement/readme-contributing-section branch from d61b90b to 336619d Compare June 13, 2016 13:27
Please follow these advices to keep the pull request workflow simple and to increase the chances of having your PR merged:

- If you add or enhance functionality, an update of *README.md* usage section should be part of the PR.
- If your PR fixes a bug you should include tests that at least fails before your code changes and pass after.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fail

@abelsoares abelsoares force-pushed the enhancement/readme-contributing-section branch 2 times, most recently from 075b443 to 4ef21a9 Compare June 13, 2016 14:12
@abelsoares abelsoares force-pushed the enhancement/readme-contributing-section branch from 4ef21a9 to 4f4a356 Compare June 13, 2016 14:13
@ricardogama ricardogama merged commit bb81512 into master Jun 13, 2016
@ricardogama ricardogama deleted the enhancement/readme-contributing-section branch June 13, 2016 14:14
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants