Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Review comments from V8 11.4.183.12 update (#678) #697

Merged
merged 1 commit into from
May 26, 2023

Conversation

ohodson
Copy link
Contributor

@ohodson ohodson commented May 24, 2023

Incorporates review comments in #678

Review comments from #678

  • Documents that patches should be generated with --full-index
  • Adds a note about git_repository shallow-since
  • Some re-wording of vs-updates.md.

Bug: EW-7428
Test: bazel build //...

@ohodson ohodson requested a review from harrishancock May 24, 2023 09:17
@ohodson ohodson force-pushed the orion/v8-11.4-review-comments branch from 1527fcf to 7657909 Compare May 24, 2023 09:23
Incorporates review comments in
#678
relating to v8-updates.md:

* Patches should be generated with --full-index
* Adds a note about shallow-since

Also includes some re-wording of v8-updates.md and
regenerates V8 patches with --full-index.

Bug: EW-7428
Test: bazel build //...
@ohodson ohodson force-pushed the orion/v8-11.4-review-comments branch from 7657909 to 4bd41fd Compare May 24, 2023 11:27
@ohodson ohodson mentioned this pull request May 25, 2023
@ohodson ohodson merged commit ef34c9a into main May 26, 2023
@ohodson ohodson deleted the orion/v8-11.4-review-comments branch May 26, 2023 09:52
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants