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

merge immutable-arraybuffer tests #4445

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

phoddie
Copy link
Contributor

@phoddie phoddie commented Apr 1, 2025

Suite of test262 tests for the Immutable ArrayBuffer proposal.

@phoddie phoddie requested a review from a team as a code owner April 1, 2025 20:44
Copy link
Contributor

@gibson042 gibson042 left a comment

Choose a reason for hiding this comment

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

@phoddie This is pretty noisy right now; please fix the lint issues:

  • Add necessary front matter fields
  • Update features.txt
  • Fix the copyrights to a single year rather than a range
  • etc.

@phoddie
Copy link
Contributor Author

phoddie commented Apr 4, 2025

Sure, will do. FWIW – "Fix the copyrights to a single year rather than a range" feels artificial.

@ljharb
Copy link
Member

ljharb commented Apr 4, 2025

It's not; it ensures that the range doesn't need to be updated every year.

@gibson042
Copy link
Contributor

Rebasing on main will also get the annotation improvements from #4449.

@phoddie
Copy link
Contributor Author

phoddie commented Apr 4, 2025

It's not; it ensures that the range doesn't need to be updated every year

??? The range only needs to be updated in years when the tests are modified, not when the calendar advances. Anyway. I will change the tests to conform with this peculiar requirement.

@phoddie phoddie requested a review from a team as a code owner April 4, 2025 17:54
@phoddie
Copy link
Contributor Author

phoddie commented Apr 4, 2025

The linter is now happy and I've attempted to resolve all comments.

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