-
Notifications
You must be signed in to change notification settings - Fork 572
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
Refactor post-install script to not depend on fs-extra #5975
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
An important fix, thanks! Just left a comment that could be good to address before merge 👍
@@ -127,8 +127,8 @@ function isAnalyticsDisabled() { | |||
} | |||
|
|||
function getRealmVersion() { | |||
const packageJsonPath = path.resolve(__dirname, "../package.json"); | |||
const packageJson = fse.readJsonSync(packageJsonPath); | |||
const packageJsonPath = path.resolve(__dirname, "..", "package.json"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
@@ -139,7 +139,7 @@ function getRealmVersion() { | |||
*/ | |||
function getRealmCoreVersion() { | |||
const dependenciesListPath = path.resolve(__dirname, "../dependencies.list"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe I deleted this file 🤔
What, How & Why?
No open issue for this.
☑️ ToDos
Compatibility
label is updated or copied from previous entryCOMPATIBILITY.md
package.json
s (if updating internal packages)Breaking
label has been applied or is not necessaryIf this PR adds or changes public API's: