Skip to content
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

support v8 revert #504

Open
gengjiawen opened this issue Oct 2, 2020 · 4 comments
Open

support v8 revert #504

gengjiawen opened this issue Oct 2, 2020 · 4 comments

Comments

@gengjiawen
Copy link
Member

gengjiawen commented Oct 2, 2020

support revert v8 commit by hash, should be usefully when test V8 function. Like nodejs/node#35415 (comment), I have a guess, but cumbersome reverting the commit in node.js repo.

@joyeecheung
Copy link
Member

Would be useful for nodejs/node-v8#175 too. We probably just need to add --reverse to the git apply command to make it work

@gengjiawen
Copy link
Member Author

Also my guess for nodejs/node#35415 (comment) is right, though I manually revert them, take quite some time (The final fix is not revert though, I found a workaround).

@github-actions
Copy link
Contributor

This issue is stale because it has been open many days with no activity. It will be closed soon unless the stale label is removed or a comment is made.

@github-actions github-actions bot added the stale label Jan 19, 2021
@gengjiawen
Copy link
Member Author

not stale.

@github-actions github-actions bot removed the stale label Jan 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants