Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

Fix the issue that date nanoseconds does not work under macOS #1434

Merged
merged 12 commits into from
Aug 12, 2019

Conversation

ultmaster
Copy link
Contributor

`eval ${localTrailConfig.command} 2>${path.join(workingDirectory, 'stderr')}`,
`echo $? \`date +%s%3N\` >${path.join(workingDirectory, '.nni', 'state')}`);
script.push(`eval ${localTrailConfig.command} 2>${path.join(workingDirectory, 'stderr')}`);
if (process.platform == 'darwin') {
Copy link
Contributor

Choose a reason for hiding this comment

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

Better to use "===" instead of "=="

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@suiguoxin suiguoxin changed the base branch from master to v1.0 August 9, 2019 03:10
@suiguoxin suiguoxin closed this Aug 12, 2019
@suiguoxin suiguoxin reopened this Aug 12, 2019
@suiguoxin suiguoxin merged commit b446cb0 into microsoft:v1.0 Aug 12, 2019
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.

7 participants