Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Traits] Support traits in swift package edit and swift package unedit commands #8417

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bripeticca
Copy link
Contributor

WIP

Fixes #8354

Motivation:

Since the edit and unedit commands can affect dependency resolution, we must support the ability to specify trait configurations.

Modifications:

  • Add entrypoint for trait options in the edit/unedit commands

Result:

TODO:

  • Determine how calls to resolve affect packages in edit mode
    • If a dependency is guarded by traits in the current workspace, can we run an edit command on it?
    • If we edit a dependency and modify its trait configurations for its dependencies, how does the resulting resolution change?
  • Add test coverage for edit/unedit cases with trait configurations
  • Add test coverage for edit/unedit cases with nested deps that are guarded by traits + how that changes with edit/unedit

- Add entrypoint for trait options in the edit/unedit
  commands

TODO:
- Determine how calls to resolve affect packages in edit mode
  - If a dependency is guarded by traits in the current workspace,
    can we run an edit command on it?
  - If we edit a dependency and modify its trait configurations
    for its dependencies, how does the resulting resolution
    change?
- Add test coverage for edit/unedit cases with trait configurations
- Add test coverage for edit/unedit cases with nested deps that
  are guarded by traits + how that changes with edit/unedit
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Traits] swift package edit and swift package unedit should support trait options
1 participant