[go: nahoru, domu]

Skip to content
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

[action] Adding no overwrite and local only options to the pod repo push command #20455

Merged
merged 1 commit into from
Jul 28, 2022

Conversation

polmum
Copy link
Contributor
@polmum polmum commented Jul 11, 2022

Checklist

  • I've run bundle exec rspec from the root directory to see all new and existing tests pass
  • I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
  • I've read the Contribution Guidelines
  • I've updated the documentation if necessary.

Motivation and Context

The PR adds new parameters to the Cocoapods command pod repo push that were missing from the Fastlane action but were present on the Cocoapods command.

Description

Two new options are being added to the pod repo push command:
no_overwrite, which disallows pushing a spec that would overwrite an existing one
local_only, which skips the step of pushing to its remote, basically acting like a dry run.

Testing Steps

I've follow the rules found on https://github.com/fastlane/fastlane/blob/master/Testing.md. I've also tested the command locally, and found no issues.

@polmum
Copy link
Contributor Author
polmum commented Jul 22, 2022

Hey
Is there anything missing from my pull request?

@polmum polmum changed the title Adding no overwrite and local only options to the pod repo push command [action] Adding no overwrite and local only options to the pod repo push command Jul 27, 2022
@getaaron
Copy link
Collaborator

Looks great, thanks for the PR!

@getaaron getaaron merged commit 850593c into fastlane:master Jul 28, 2022
@polmum polmum deleted the pod-repo-push-add-options branch July 28, 2022 12:49
@fastlane-bot
Copy link

Hey @polmum 👋

Thank you for your contribution to fastlane and congrats on getting this pull request merged 🎉
The code change now lives in the master branch, however it wasn't released to RubyGems yet.
We usually ship about once a week, and your PR will be included in the next one.

Please let us know if this change requires an immediate release by adding a comment here 👍
We'll notify you once we shipped a new release with your changes 🚀

Copy link
@fastlane-bot fastlane-bot left a 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.209.0 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants