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

Implement new syntax. Closes #1. #5

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Implement new syntax. Closes #1. #5

wants to merge 4 commits into from

Conversation

snanovskyi
Copy link

Here is my implementation, as there is no progress in #4.
@aqrln @primeare, check it please.

@primeare
Copy link
Member

@sun1x ok, I'll review it in a few days.

@@ -1 +1,2 @@
Timur Shemsedinov <[email protected]>
Sergey Nanovsky <[email protected]>
Copy link
Member

Choose a reason for hiding this comment

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

Add @primeare to AUTHORS

Copy link
Member

Choose a reason for hiding this comment

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

That would not be related to this PR :)
I am okay with @sun1x adding himself to the list in his first PR, though I would not say it is a good thing to do generally. But adding other people should definitely be done separately.

@@ -0,0 +1,48 @@
/* eslint-disable no-template-curly-in-string */
Copy link
Member

Choose a reason for hiding this comment

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

Why underscores __tests__ ?

Copy link
Member

Choose a reason for hiding this comment

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

That's how it is usually done in Jest, but it is not a requirement. If I recall correctly, you may as well name the directory as you wish and just name your test files as *.test.js or something like that.

Copy link
Author

Choose a reason for hiding this comment

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

Should I change it?
By default, Jest uses this regex to detect test files.

@aqrln
Copy link
Member

aqrln commented May 18, 2017

I've just enabled Travis for this repo, so I'm going to close and reopen this PR to trigger a build.

@aqrln aqrln closed this May 18, 2017
@aqrln aqrln reopened this May 18, 2017
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.

4 participants