Skip to content
This repository has been archived by the owner on Oct 15, 2020. It is now read-only.

master branch cleanup #466

Merged
merged 2 commits into from
Feb 16, 2018
Merged

master branch cleanup #466

merged 2 commits into from
Feb 16, 2018

Conversation

kfarnung
Copy link
Contributor

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

chakrashim, src, test

@kfarnung kfarnung requested a review from MSLaguana February 14, 2018 22:30
@kfarnung
Copy link
Contributor Author

@kfarnung kfarnung requested a review from boingoing February 14, 2018 22:31
@kfarnung
Copy link
Contributor Author

I'll rebase this on #465 once that lands.

@kfarnung kfarnung self-assigned this Feb 14, 2018
Copy link
Contributor

@boingoing boingoing left a comment

Choose a reason for hiding this comment

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

LGTM

@kfarnung
Copy link
Contributor Author

kfarnung commented Feb 15, 2018

@kfarnung kfarnung force-pushed the cleanup branch 3 times, most recently from 6e6e6bf to 0989d78 Compare February 16, 2018 02:10
kfarnung added a commit to kfarnung/node-chakracore that referenced this pull request Feb 16, 2018
* Fixed differences with upstream in the `src/` folder
* Fixed linux test failure in `test-cli-node-options.js`

PR-URL: nodejs#466
Reviewed-By: Taylor Woll <tawoll@ntdev.microsoft.com>
Reviewed-By: Hitesh Kanwathirtha <hiteshk@microsoft.com>
kfarnung added a commit to kfarnung/node-chakracore that referenced this pull request Feb 16, 2018
* Moved `unordered_set` to `v8-tracing.h` to match v8
* Fixed `int64`->`double` implicit cast in `v8-platform.h`

PR-URL: nodejs#466
Reviewed-By: Taylor Woll <tawoll@ntdev.microsoft.com>
Reviewed-By: Hitesh Kanwathirtha <hiteshk@microsoft.com>
* Fixed differences with upstream in the `src/` folder
* Fixed linux test failure in `test-cli-node-options.js`

PR-URL: nodejs#466
Reviewed-By: Taylor Woll <tawoll@ntdev.microsoft.com>
Reviewed-By: Hitesh Kanwathirtha <hiteshk@microsoft.com>
* Moved `unordered_set` to `v8-tracing.h` to match v8
* Fixed `int64`->`double` implicit cast in `v8-platform.h`

PR-URL: nodejs#466
Reviewed-By: Taylor Woll <tawoll@ntdev.microsoft.com>
Reviewed-By: Hitesh Kanwathirtha <hiteshk@microsoft.com>
@kfarnung kfarnung merged commit 3aff817 into nodejs:master Feb 16, 2018
@kfarnung kfarnung deleted the cleanup branch February 16, 2018 22:47
kfarnung added a commit to kfarnung/node-chakracore that referenced this pull request Feb 23, 2018
* Moved `unordered_set` to `v8-tracing.h` to match v8
* Fixed `int64`->`double` implicit cast in `v8-platform.h`

PR-URL: nodejs#466
Reviewed-By: Taylor Woll <tawoll@ntdev.microsoft.com>
Reviewed-By: Hitesh Kanwathirtha <hiteshk@microsoft.com>
kfarnung added a commit to kfarnung/node-chakracore that referenced this pull request Mar 6, 2018
* Moved `unordered_set` to `v8-tracing.h` to match v8
* Fixed `int64`->`double` implicit cast in `v8-platform.h`

PR-URL: nodejs#466
Reviewed-By: Taylor Woll <tawoll@ntdev.microsoft.com>
Reviewed-By: Hitesh Kanwathirtha <hiteshk@microsoft.com>
kfarnung added a commit to kfarnung/node-chakracore that referenced this pull request Mar 7, 2018
* Moved `unordered_set` to `v8-tracing.h` to match v8
* Fixed `int64`->`double` implicit cast in `v8-platform.h`

PR-URL: nodejs#466
Reviewed-By: Taylor Woll <tawoll@ntdev.microsoft.com>
Reviewed-By: Hitesh Kanwathirtha <hiteshk@microsoft.com>
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.

3 participants