Skip to content

Converge command line tools into one: swift-java #248

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 8 commits into from
Jun 9, 2025

Conversation

ktoso
Copy link
Collaborator

@ktoso ktoso commented Jun 5, 2025

Work in progress.

We want to have one command line tool to rule them all™ :-)

@ktoso ktoso force-pushed the wip-converge-cli branch from 9f898e4 to 65400da Compare June 6, 2025 11:37
@ktoso
Copy link
Collaborator Author

ktoso commented Jun 6, 2025

This should be complete but I need to give it a rebase now

@ktoso ktoso force-pushed the wip-converge-cli branch from bc7d448 to 896b2be Compare June 9, 2025 06:49
@ktoso ktoso changed the title [WIP] Converge command line tools into one: swift-java Converge command line tools into one: swift-java Jun 9, 2025
@ktoso
Copy link
Collaborator Author

ktoso commented Jun 9, 2025

This just moves code around so we have one command line tool: swift-java that has various modes.

There will be follow up PRs further cleaning up and making it easier to use each of the modes. Merging so we can keep iterating and rebase other PRs waiting.

@ktoso ktoso marked this pull request as ready for review June 9, 2025 07:02
@ktoso ktoso requested a review from rintaro June 9, 2025 07:03
@ktoso
Copy link
Collaborator Author

ktoso commented Jun 9, 2025

post-merge review welcome, there will be more cleanups incoming here. Some commands need to be renamed still and I think we need to make them commands swift-java resolve rather than --fetch etc.

@ktoso ktoso merged commit 569a283 into swiftlang:main Jun 9, 2025
17 checks passed
@ktoso ktoso deleted the wip-converge-cli branch June 9, 2025 07:03
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.

1 participant