Skip to content

Conversation

podtserkovskiy
Copy link
Contributor

@podtserkovskiy podtserkovskiy commented Aug 6, 2025

Closes #272

@podtserkovskiy
Copy link
Contributor Author

cc @JamyDev

@JamyDev
Copy link
Contributor

JamyDev commented Aug 7, 2025

Seems sane to me. Tagging @sywhang for review.

@podtserkovskiy podtserkovskiy force-pushed the unify_package_and_archive_step2 branch from fd4fd0f to 030b99f Compare August 12, 2025 23:25
@podtserkovskiy
Copy link
Contributor Author

podtserkovskiy commented Aug 12, 2025

@sywhang @linzhp Could you please review this PR?
It also unblocks Go1.25 compatibility golang/go#74462

I have two commits to simplify review

  • step1: re-uses "package" implementation for archive by removing receiver for methods
  • step2: makes both "package" and "archive" to work with ExportFile + upgrades x/tools for modern compiler support

@podtserkovskiy podtserkovskiy force-pushed the unify_package_and_archive_step2 branch from 030b99f to f142e57 Compare August 19, 2025 11:11
@podtserkovskiy
Copy link
Contributor Author

Hey folks, is there a chance to get this PR merged?

Copy link
Contributor

@sywhang sywhang left a comment

Choose a reason for hiding this comment

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

Thanks for this PR, and sorry for the delay in response. The Uber Go team is fairly occupied lately so we've got a bit of catch up reviews to do.

Just took a look at this PR and it looks good, but there are some behavioral changes in archive mode that's being "fixed" as a result of the code merging that might be worth adding to CHANGELOG:

  • we now error out on type constraints in archive mode.
  • we now handle type aliases explicitly in archive mode.

Feel free to modify CHANGELOG, otherwise I plan on merging this tomorrow morning and will be adding these ourselves later before releasing.

@podtserkovskiy podtserkovskiy force-pushed the unify_package_and_archive_step2 branch from f142e57 to 9098d3b Compare October 3, 2025 10:39
@podtserkovskiy
Copy link
Contributor Author

No worries, thank you for the review. I've added information about this change to the CHANGELOG.

Copy link
Contributor

@sywhang sywhang left a comment

Choose a reason for hiding this comment

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

thanks!

@sywhang sywhang merged commit ca5300b into uber-go:main Oct 4, 2025
5 checks passed
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.

Unify package and archive modes

4 participants