-
Notifications
You must be signed in to change notification settings - Fork 821
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
Add support for adding build tool plugins to targets #1374
Add support for adding build tool plugins to targets #1374
Conversation
575ec5e
to
7ca1234
Compare
1f02625
to
c93a406
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @BarredEwe! Looks good, left a few comments. Could you also please add:
- a changelog entry https://github.com/yonaskolb/XcodeGen/blob/master/CHANGELOG.md
- an update in https://github.com/yonaskolb/XcodeGen/blob/master/Docs/ProjectSpec.md
- an example usage in the test project (which serves as an snapshot and integration test) https://github.com/yonaskolb/XcodeGen/blob/master/Tests/Fixtures/TestProject/project.yml
6eef473
to
96130d1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job! Let's wait @yonaskolb 's final review and XcodeProj's release
a676871
to
d4ce70f
Compare
Hi @yonaskolb and @freddi-kit ! Cheers, |
@yanniks this feature is good to go, we're just waiting on a release of XcodeProj (And the recent merge conflicts to be resolved) |
b02708d
to
9e3939d
Compare
Co-authored-by: freddi(Yuki Aki) <[email protected]>
Co-authored-by: freddi(Yuki Aki) <[email protected]>
Co-authored-by: freddi(Yuki Aki) <[email protected]>
Co-authored-by: freddi(Yuki Aki) <[email protected]>
616e666
to
b9d7d2a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for all your great work on this @BarredEwe! 🙏
Was there a dependency on Xcode version for this to work? Works fine on my machine with Xcode 14.3.1 but seems to not link it up correctly on my colleague's machine which is on Xcode 14.2 |
@OmranK Can you add an Xcode error? PS: If you add a plugin without XcodeGen in Xcode 14.2, does it work correctly? |
@BarredEwe There is no error. It simply shows "Run Build Tool Plug-ins (0 items)". If he manually clicks + and adds in SwiftLintPlugin, it works. |
@OmranK I can't reproduce this on Xcode 14.2 and the project Fixtures/SPM. |
Short description 📝
Support for Build Tool Plugins has been added here. A detailed discussion can be read here: #1290
Details ⚙️
API
To use plugins, you need to specify in your target which plugin you want to connect.
And don't forget to connect the package.
What is generated
PBXTargetDependency
:plugin:
) is added:What is being validated
If you add a plugin that references a package that is not in the project:
There will be an error:
Spec validation error: Plugin PrefirePlaybookPlugin has invalide package reference SomeName