Skip to content

Conversation

devsnek
Copy link
Member

@devsnek devsnek commented Jun 28, 2024

  • Symbol.asyncDispose no longer needs to be polyfilled
  • assorted updates for cppgc api changes

@devsnek devsnek added the ci-draft Run the CI on draft PRs. label Jun 28, 2024
Comment on lines +66 to +70
impl deno_core::GarbageCollected for ContextifyContext {
fn trace(&self, visitor: &v8::cppgc::Visitor) {
visitor.trace(&self.context);
visitor.trace(&self.sandbox);
}
Copy link
Member

Choose a reason for hiding this comment

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

👍

@devsnek devsnek force-pushed the deno-core-upgrade branch from 30d04c6 to 6db1f3a Compare July 1, 2024 15:56
@devsnek devsnek changed the title chore: upgrade deno_core feat: upgrade deno_core Jul 1, 2024
@devsnek devsnek force-pushed the deno-core-upgrade branch from 6db1f3a to 06d17b6 Compare July 1, 2024 16:13
@devsnek devsnek marked this pull request as ready for review July 1, 2024 16:19
@devsnek devsnek force-pushed the deno-core-upgrade branch from 06d17b6 to 159ccff Compare July 1, 2024 18:00
Copy link
Member

@nathanwhit nathanwhit left a comment

Choose a reason for hiding this comment

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

LGTM

@devsnek devsnek merged commit a555cb4 into main Jul 1, 2024
@devsnek devsnek deleted the deno-core-upgrade branch July 1, 2024 22:48
sbmsr pushed a commit to sbmsr/deno-1 that referenced this pull request Jul 2, 2024
- Symbol.asyncDispose no longer needs to be polyfilled
- assorted updates for cppgc api changes
zebreus pushed a commit to zebreus/deno that referenced this pull request Jul 8, 2024
- Symbol.asyncDispose no longer needs to be polyfilled
- assorted updates for cppgc api changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-draft Run the CI on draft PRs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants