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
[spaceship] Remove multi_xml dependency #16697
[spaceship] Remove multi_xml dependency #16697
Conversation
This was an optional dependency for `faraday_middleware` when parsing XML responses. As far as I can tell, and from what is being tested with fixtures, that used to be the case but is not any longer using the App Store Connect API. There is a custom plist middleware, but that one is in fastlane itself and does not make use of `multi_xml`. I believe it's used by `#provisioning_profiles_via_xcode_api`. https://github.com/fastlane/fastlane/blob/2.150.0.rc4/spaceship/lib/spaceship/portal/portal_client.rb#L653-L674 https://github.com/fastlane/fastlane/blob/2.150.0.rc4/spaceship/lib/spaceship/helper/plist_middleware.rb
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.
This is great! I ran across this the other day and wasn't actually sure if it was used anymore but it sure doesn't 😇 Thanks for taking the time to do the research!
Hey @aried3r 👋 Thank you for your contribution to fastlane and congrats on getting this pull request merged 🎉 Please let us know if this change requires an immediate release by adding a comment here 👍 |
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.
Congratulations! 🎉 This was released as part of fastlane 2.151.0 🚀
This was an optional dependency for `faraday_middleware` when parsing XML responses. As far as I can tell, and from what is being tested with fixtures, that used to be the case but is not any longer using the App Store Connect API. There is a custom plist middleware, but that one is in fastlane itself and does not make use of `multi_xml`. I believe it's used by `#provisioning_profiles_via_xcode_api`. https://github.com/fastlane/fastlane/blob/2.150.0.rc4/spaceship/lib/spaceship/portal/portal_client.rb#L653-L674 https://github.com/fastlane/fastlane/blob/2.150.0.rc4/spaceship/lib/spaceship/helper/plist_middleware.rb
Checklist
bundle exec rspec
from the root directory to see all new and existing tests passbundle exec rubocop -a
to ensure the code style is validMotivation and Context
I believe
multi_xml
is not needed. This PR is what #16664 should have been. Thanks to the feedback by @janpio I took another look.Description
This was an optional dependency for
faraday_middleware
when parsing XML responses. As far as I can tell, and from what is being tested with fixtures, that used to be the case but is not any longer, using the App Store Connect API.There is a custom plist middleware, but that one is in fastlane itself and does not make use of
multi_xml
. I believe it's used by#provisioning_profiles_via_xcode_api
.https://github.com/fastlane/fastlane/blob/2.150.0.rc4/spaceship/lib/spaceship/portal/portal_client.rb#L653-L674
https://github.com/fastlane/fastlane/blob/2.150.0.rc4/spaceship/lib/spaceship/helper/plist_middleware.rb
The client was introduced in #14888.
Testing Steps
Ran the tests, worked. :)