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

Add support for location scenario reference #813

Merged
merged 3 commits into from
Mar 23, 2021

Conversation

freak4pc
Copy link
Contributor

@freak4pc freak4pc commented Mar 8, 2021

This PR parses a launch action's location scenario reference - the portion which defines the simulated GPS location of a runnable scheme

I did my best to assume what are the right tests here, but I'm not a ruby developer so if I missed anything please do let me know :)

Thanks!

@freak4pc freak4pc force-pushed the location_scenario_reference branch 2 times, most recently from 3186da0 to 84d9323 Compare March 9, 2021 07:36
@freak4pc
Copy link
Contributor Author

Sorry for the ping but any chance to look at this? We want to add some testing to a different OSS project that needs this reference exposed. @dnkoutso 🙏

@freak4pc
Copy link
Contributor Author

ready for you when you can :) @dnkoutso

@freak4pc
Copy link
Contributor Author

Any chance to get this pulled in to unblock a different PR waiting for it? @amorde @dnkoutso

@igor-makarov
Copy link
Contributor

LGTM!
@segiddins @dnkoutso?

@freak4pc
Copy link
Contributor Author

The approval isn't the issue (you already did that) I just don't have merging permissions unfortunately :)

@dnkoutso

@dnkoutso dnkoutso merged commit 907c817 into CocoaPods:master Mar 23, 2021
@freak4pc freak4pc deleted the location_scenario_reference branch March 23, 2021 17:46
@freak4pc
Copy link
Contributor Author

Thank you! Any plans on cutting a release soon? Seems like the last one was 6 months ago, based on GitHub at least?

@connor-ricks
Copy link
Contributor

Also hoping to get a release soon as well! I would love to start using a few things that have been added since the last release.

@dnkoutso
Copy link
Contributor

dnkoutso commented Jul 6, 2021

released with 1.20.0

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

Successfully merging this pull request may close these issues.

4 participants