Skip to content
This repository has been archived by the owner on Nov 14, 2018. It is now read-only.

Update README.md #9

Merged
merged 1 commit into from
Nov 13, 2017
Merged

Update README.md #9

merged 1 commit into from
Nov 13, 2017

Conversation

SergeAstapov
Copy link
Contributor

Add note about existsSync being un-deprecated.

README.md Outdated
@@ -2,10 +2,10 @@ exists-sync
===========
[![Build Status](https://travis-ci.org/ember-cli/exists-sync.svg)](https://travis-ci.org/ember-cli/exists-sync) [![Build status](https://ci.appveyor.com/api/projects/status/c05xyb4s80pn66yo?svg=true)](https://ci.appveyor.com/project/embercli/exists-sync)

Replacement for [fs.existsSync()](https://nodejs.org/api/fs.html#fs_fs_exists_path_callback), which is being deprecated:
Replacement for [fs.existsSync()](https://nodejs.org/api/fs.html#fs_fs_exists_path_callback) (which has been un-deprecated in Node.js since [6.8.0](https://nodejs.org/en/blog/release/v6.8.0/)):
Copy link
Contributor

Choose a reason for hiding this comment

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

I would link to PR

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks @twokul! Updated! Makes sense - PR has much more information.

Add note about `existsSync` being un-deprecated.
@twokul twokul merged commit 60f0e34 into ember-cli:master Nov 13, 2017
@twokul
Copy link
Contributor

twokul commented Nov 13, 2017

thanks!

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

Successfully merging this pull request may close these issues.

2 participants