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

Sync parse functions #28

Merged
merged 9 commits into from Oct 11, 2015
Merged

Sync parse functions #28

merged 9 commits into from Oct 11, 2015

Conversation

seka
Copy link
Contributor

@seka seka commented Sep 21, 2015

Expose functions parseSync and parseFromFilesSync.

And added those test.

@@ -0,0 +1,15 @@
[*]
Copy link
Member

Choose a reason for hiding this comment

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

The root .editorconfig file already has these settings.

@seka
Copy link
Contributor Author

seka commented Sep 22, 2015

thank you for review.
It was wrong to added a test/.editorconfig for a test.
I retouch it to use an existing .editorconfig.

@jednano
Copy link
Member

jednano commented Sep 22, 2015

Can you add some readme docs for these new methods?

@seka
Copy link
Contributor Author

seka commented Sep 23, 2015

sorry... I am not confident that I will explain that well.

@jednano
Copy link
Member

jednano commented Sep 23, 2015

If you're worried about the English, I can correct the spelling/grammar for you, but this really needs some readme info and I think you're the best to add it.

@seka
Copy link
Contributor Author

seka commented Sep 23, 2015

ok! I'll try it.

@seka
Copy link
Contributor Author

seka commented Oct 10, 2015

@jedmao
I was waiting for you. What happened?

jednano pushed a commit that referenced this pull request Oct 11, 2015
@jednano jednano merged commit b2e00d9 into editorconfig:master Oct 11, 2015
@jednano
Copy link
Member

jednano commented Oct 11, 2015

Sorry @seka ! Must have missed the notification. It's merged in and published as 0.13.0. Thanks!

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.

None yet

2 participants