Skip to content

Remove usage of private fields #314

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

Merged
merged 1 commit into from
Mar 20, 2025
Merged

Remove usage of private fields #314

merged 1 commit into from
Mar 20, 2025

Conversation

ptomato
Copy link
Contributor

@ptomato ptomato commented Mar 19, 2025

Downleveling this with the TypeScript compiler would add a runtime dependency on tslib. The TimeDuration object is private anyway and not exported to client code, so we don't strictly need this method to be private.

@ptomato ptomato requested review from justingrant and 12wrigja March 19, 2025 21:10
@ptomato
Copy link
Contributor Author

ptomato commented Mar 19, 2025

Looks like CI's node 20.x got upgraded from 20.18 to 20.19, that brought in a new CLDR and broke a test. #315 adds a new expected failures file until we can update to a version of test262 that takes CLDR 46 into account.

Copy link
Contributor

@justingrant justingrant left a comment

Choose a reason for hiding this comment

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

👍

Downleveling this with the TypeScript compiler would add a runtime
dependency on tslib. The TimeDuration object is private anyway and not
exported to client code, so we don't strictly need this method to be
private.
@ptomato ptomato force-pushed the remove-tslib-dependency branch from 7f55e00 to 060fe7f Compare March 20, 2025 02:12
@ptomato ptomato merged commit 9934a0f into main Mar 20, 2025
19 checks passed
@ptomato ptomato deleted the remove-tslib-dependency branch March 20, 2025 02:15
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.

2 participants