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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add synchronous option to Trunk push #1551

Merged
merged 1 commit into from Jun 9, 2020

Conversation

Jake-Prickett
Copy link
Contributor

Add synchronous option to trunk push in podspec script

Motivation:

Currently the build_podspec.sh script needs to be ran multiple times due to Podspecs not immediately becoming available for sequential Podspecs. This has been an ongoing issue where I have to run the script multiple times manually in order to get all of them through.

Modifications:

Add CocoaPods/cocoapods-trunk#147 to the script

Result:

Now, the script will only need to be ran once! 馃帀

@swift-server-bot
Copy link

Can one of the admins verify this patch?

10 similar comments
@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

Copy link
Member

@weissi weissi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't really know what --synchronous does but you're the main user of this script :)

@weissi
Copy link
Member

weissi commented Jun 9, 2020

@Jake-Prickett sorry about the previous comment, your motivation explains it really well. Thank you!

@weissi
Copy link
Member

weissi commented Jun 9, 2020

@swift-nio-bot add to whitelist

@Lukasa Lukasa added the patch-version-bump-only For PRs that when merged will only cause a bump of the patch version, ie. 1.0.x -> 1.0.(x+1) label Jun 9, 2020
@Lukasa Lukasa added this to the 2.18.0 milestone Jun 9, 2020
@Lukasa Lukasa merged commit c05eeb4 into apple:master Jun 9, 2020
@Jake-Prickett
Copy link
Contributor Author

haha no worries @weissi! Just discovered this yesterday and it'll make life a bit easier for whoever runs it!

@Jake-Prickett Jake-Prickett deleted the add-synchronous-option-for-pods branch June 9, 2020 13:52
pull bot pushed a commit to scope-demo/swift-nio that referenced this pull request Jun 9, 2020
Co-authored-by: Prickett <jpricke1@ford.com>
glbrntt added a commit to glbrntt/swift-nio-http2 that referenced this pull request Jun 10, 2020
glbrntt added a commit to glbrntt/swift-nio-ssl that referenced this pull request Jun 10, 2020
Lukasa pushed a commit to apple/swift-nio-http2 that referenced this pull request Jun 10, 2020
Lukasa pushed a commit to apple/swift-nio-ssl that referenced this pull request Jun 10, 2020
@Lukasa Lukasa modified the milestones: 2.18.0, 2.19.0 Jun 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch-version-bump-only For PRs that when merged will only cause a bump of the patch version, ie. 1.0.x -> 1.0.(x+1)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants