Skip to content

Revert "hotfix(file): cast dev to uint32" - #2889

Merged
NoahDragon merged 2 commits into
masterfrom
revert-a2184229
Dec 16, 2017
Merged

NoahDragon merged 2 commits into
masterfrom
revert-a2184229

Conversation

@JLHwung

@JLHwung JLHwung commented Dec 6, 2017 •

Copy link
Copy Markdown
Member

This reverts commit a218422.

Since nodejs/node#16705 is merged, the hotfix is unecessary.

Conflicts:

test/scripts/box/file.js

  • Passed the CI test.

Todo

  • Hold this PR until appveyor update node.js to 6.12.1

@coveralls

coveralls commented Dec 6, 2017 •

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 97.226% when pulling 42e7c571faf4f82a9314ee5809fefc0981c9166d on revert-a2184229 into dee3b03 on master.

@coveralls

coveralls commented Dec 7, 2017 •

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 97.226% when pulling 62df5a78ea1144a8f7094a31ce9fc0b90d1de646 on revert-a2184229 into dee3b03 on master.

@coveralls

coveralls commented Dec 8, 2017 •

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 97.226% when pulling 0b66c2ebbb69923bc0c48a21d9d4977951759b96 on revert-a2184229 into dee3b03 on master.

@coveralls

coveralls commented Dec 8, 2017 •

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 97.226% when pulling b14c326f684a0f54513be9581fc8e8e9e0273473 on revert-a2184229 into dee3b03 on master.

This reverts commit a218422.

Since nodejs/node#16705 is merged, the hotfix is unecessary.
@coveralls

coveralls commented Dec 10, 2017 •

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 97.226% when pulling 487c702 on revert-a2184229 into dee3b03 on master.

@coveralls

coveralls commented Dec 10, 2017 •

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 97.226% when pulling 0ea508b on revert-a2184229 into dee3b03 on master.

@NoahDragon NoahDragon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Awesome! This issue bugs me a long time.

@NoahDragon
NoahDragon merged commit d4b4470 into master Dec 16, 2017
@NoahDragon
NoahDragon deleted the revert-a2184229 branch December 16, 2017 02:15
thom4parisot referenced this pull request in thom4parisot/hexo Jan 17, 2020
* Revert "hotfix(file): cast dev to uint32"

This reverts commit a218422.

Since nodejs/node#16705 is merged, the hotfix is unecessary.

* test(appveyor): update to exact node.js version
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.

3 participants