Skip to content
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

Two new styles added for the table #58

Merged
merged 4 commits into from
Dec 15, 2018
Merged

Conversation

edzcodes
Copy link

No description provided.

@tdwright tdwright mentioned this pull request Oct 28, 2018
@tdwright
Copy link
Owner

Hi @ErsinDagli,

Thanks for this contribution. 😃

In order to maintain a high level of test coverage, we ask that all newly contributed code is accompanied by new tests.

Would you mind adding tests for these new styles please?

Thanks again,
Tom

@tdwright tdwright changed the base branch from master to develop November 6, 2018 00:53
@ghost ghost assigned tdwright Nov 6, 2018
@ghost ghost added the review label Nov 6, 2018
@tdwright
Copy link
Owner

Hi @ErsinDagli

How's it going? Any progress with these tests?

They should be very easy to write - you'd just need to use the existing tests as a template. So please don't be put off!

If you feel like it, there's also a new suggested style too. See #61

Tom

@tdwright tdwright added this to the Version 2.0 milestone Nov 18, 2018
@tdwright tdwright assigned edzcodes and unassigned tdwright Nov 21, 2018
@tdwright tdwright self-requested a review November 21, 2018 10:43
@tdwright tdwright merged commit 6765ca4 into tdwright:develop Dec 15, 2018
@ghost ghost removed the review label Dec 15, 2018
@tdwright
Copy link
Owner

I've added these tests and have merged these styles in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants