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
Fix build phases not being generated in the right positions #506
Conversation
And I have a working directory | ||
Then I copy the fixture ios_app_with_actions into the working directory | ||
Then tuist generates the project | ||
Then the target App should have the build phase Tuist in the first position |
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.
@parse am I right defining in this test what you expect in the generated project?
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.
Correct!
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.
Awesome! With this test in place we'll make sure we don't introduce regressions in the future.
SwiftLint found issuesWarnings
Generated by π« Danger |
51c9330
to
928eac6
Compare
Codecov Report
@@ Coverage Diff @@
## master #506 +/- ##
==========================================
+ Coverage 92.17% 92.18% +0.01%
==========================================
Files 352 352
Lines 18450 18461 +11
==========================================
+ Hits 17006 17018 +12
+ Misses 1444 1443 -1
Continue to review full report at Codecov.
|
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.
π
Fixes #505
Short description π
As @parse reported, the target actions were not generated in the right position. That happens because after calling the build phases generator, we call the link phases generator which might insert phases after the post actions.
Solution π¦
Move the calls to generate the target actions to the
TargetGenerator
.Implementation π©βπ»π¨βπ»