Skip to content

Conversation

@bakkot
Copy link
Member

@bakkot bakkot commented Jul 8, 2024

Tests for tc39/proposal-arraybuffer-base64#60 and tc39/proposal-arraybuffer-base64#58, which got consensus after the proposal had achieved stage 3 based on user and implementer feedback.

Based on #3994 (which incidentally is ready).

@bakkot bakkot requested a review from a team as a code owner July 8, 2024 05:11
@ljharb ljharb requested a review from a team as a code owner July 8, 2024 16:31
Base automatically changed from base64-tests to main July 8, 2024 16:56
Copy link
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

The "writes up to error" tests fail on my polyfill, so I'll review those after trying to fix my bug, but otherwise they look OK. omit-padding tests pass for me.

Copy link
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

confirmed, once i updated my implementation to match the updated spec, tests all pass.

@ljharb ljharb requested a review from a team July 8, 2024 18:04
ljharb added a commit to es-shims/es-arraybuffer-base64 that referenced this pull request Jul 8, 2024
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.

4 participants