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

Fixes increment_build_number_in_xcodeproj in xocde 11 #48

Merged
merged 3 commits into from
Mar 31, 2020

Conversation

prathmeshranaut
Copy link
Contributor

Error

23:35:08: --- Step: increment_build_number_in_xcodeproj ---
23:35:08: -------------------------------------------------
[23:35:09]: Unknown method 'plist'
[23:35:09]: To call another action from an action use other_action.plist instead
+------+-------------------------------------+-------------+
| fastlane summary |
+------+-------------------------------------+-------------+
| Step | Action | Time (in s) |
+------+-------------------------------------+-------------+
| 馃挜 | increment_build_number_in_xcodeproj | 0 |
+------+-------------------------------------+-------------+
[23:35:09]: fastlane finished with errors
[!] To call another action from an action use other_action.plist instead

Fastfile to reproduce

lane :increment do
  increment_build_number_in_xcodeproj(
    xcodeproj: '<your-xcodeproject>',
    target: <target>,
    build_configuration_name: <Release>
  )
end

Fix

Looks like an incorrect comparison. The plugin works fine after the change.

@prathmeshranaut
Copy link
Contributor Author

Fixes #47

@giacomoRanieri
Copy link

giacomoRanieri commented Feb 20, 2020

Can someone accept the pull request and release a new version? I need the fix for my project.

@jdouglas-nz
Copy link
Collaborator

@giacomoRanieri for now could you pls just target @prathmeshranaut 's fork? I'm grateful for his work but I'd love for him to add a test to make sure we never regress this again. I will look at this tonight or tomorrow if he hasn't already馃

@@ -87,7 +87,7 @@ def self.set_version_number_using_target(params, next_version_number)
project.save
end

def self.set_build_version_using_scheme(params, next_version_number)
def self.set_version_number_using_scheme(params, next_version_number)
Copy link
Collaborator

Choose a reason for hiding this comment

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

weren't there any unit tests that were testing the old -- albeit incorrect - method ? do you still have a green light?

@jdouglas-nz jdouglas-nz merged commit ffaf055 into SiarheiFedartsou:master Mar 31, 2020
@jdouglas-nz
Copy link
Collaborator

@prathmeshranaut / @giacomoRanieri / @shuima101 I have released this fix. please give it a go and tell me how you go. Apologies for the very slow response time on this one.

@KeiroMidori
Copy link

Works for me. Thanks!

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.

None yet

4 participants