Skip to content

Update ixdtf to icu4x main git branch#365

Merged
nekevss merged 13 commits into
boa-dev:mainfrom
robot-head:ms/update_ixdtf
Jul 4, 2025
Merged

Update ixdtf to icu4x main git branch#365
nekevss merged 13 commits into
boa-dev:mainfrom
robot-head:ms/update_ixdtf

Conversation

@robot-head

@robot-head robot-head commented Jun 22, 2025

Copy link
Copy Markdown
Contributor

Possibly fixes #230 and #226

@jedel1043

Copy link
Copy Markdown
Member

@nekevss We would have to make a release before merging this. Otherwise, we may need to wait until a new version of ixdtf gets published.

@nekevss

nekevss commented Jun 22, 2025

Copy link
Copy Markdown
Member

Yeah, it's been two weeks as is, so a release would probably be useful.

So we should merge in what FFI we can, make a release, and then we can merge this. I don't expect the ixdtf release to be too far behind. I'll see about putting up PRs for ixdtf and resb sometime this week.

@nekevss

nekevss commented Jul 3, 2025

Copy link
Copy Markdown
Member

@robot-head There's a new release of ixdtf. Would you be able to update this PR to the new version vs. using ICU4X's repository?

@robot-head

Copy link
Copy Markdown
Contributor Author

@robot-head There's a new release of ixdtf. Would you be able to update this PR to the new version vs. using ICU4X's repository?

Done!

@nekevss nekevss left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks like this just needs a rebase / merge 😄 But looks good to me overall!

Just to note: this won't technically fix either of the issues attached, but it is a good baseline update overall.

To address, #226 the code on line 55 of parsers/timezone.rs would need to be update to the IXDTF timezone parser. Primarily to use TimeZoneParser::parse_identifier.

If you'd like to update on this PR, feel free. If not, that's fine too. This is still a good step forward!

@nekevss nekevss mentioned this pull request Jul 4, 2025
@robot-head

Copy link
Copy Markdown
Contributor Author

Looks like this just needs a rebase / merge 😄 But looks good to me overall!

Just to note: this won't technically fix either of the issues attached, but it is a good baseline update overall.

To address, #226 the code on line 55 of parsers/timezone.rs would need to be update to the IXDTF timezone parser. Primarily to use TimeZoneParser::parse_identifier.

If you'd like to update on this PR, feel free. If not, that's fine too. This is still a good step forward!

I believe this is now done!

@robot-head robot-head requested a review from nekevss July 4, 2025 02:10

@nekevss nekevss left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Awesome. Looks good to me!

@nekevss nekevss merged commit 5773df3 into boa-dev:main Jul 4, 2025
8 checks passed
@robot-head robot-head deleted the ms/update_ixdtf branch July 4, 2025 02:28
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.

UtcOffset should reject when the offset is subminute precision.

3 participants