Skip to content

Commit 84c2fca

Browse files
committed
Add Alternatives Considered section for swift migrate vs. swift package migrate
1 parent a87b818 commit 84c2fca

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

proposals/NNNN-adoption-tooling-for-swift-features.md

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# Migration tooling for Swift features
22

33
* Proposal: [SE-NNNN](NNNN-filename.md)
4-
* Authors: [Anthony Latsis](https://github.com/AnthonyLatsis)
4+
* Authors: [Anthony Latsis](https://github.com/AnthonyLatsis), [Pavel Yaskevich](https://github.com/xedin)
55
* Review Manager: TBD
66
* Status: **Awaiting implementation**
77
* Implementation: TBD
@@ -344,6 +344,12 @@ The next candidates in line per discussions are ***adopt***, ***audit***,
344344
or rules.
345345
To illustrate, this mode could appropriately suggest switching from `any P`
346346
to `some P` for `ExistentialAny`.
347+
348+
### `swift migrate` vs. `swift package migrate`
349+
350+
Rather than have migrate as a subcommand (ie. `swift package migrate`), another option is add another top level command, ie. `swift migrate`.
351+
352+
As the command applies to the current package, we feel a `swift package` sub-command fits better than a new top-level command. This also aligns with the recently added package refactorings (eg. `add-target`).
347353

348354
## Acknowledgements
349355

0 commit comments

Comments
 (0)