Skip to content

Replace image describing post-build script#7

Merged
BalestraPatrick merged 4 commits intoMobileNativeFoundation:masterfrom
codedbypm:master
Jun 15, 2019
Merged

Replace image describing post-build script#7
BalestraPatrick merged 4 commits intoMobileNativeFoundation:masterfrom
codedbypm:master

Conversation

@codedbypm
Copy link
Contributor

@codedbypm codedbypm commented Jun 12, 2019

This corrects the wrong image in the README.md regarding XCode integration.

post-action-run-script

@BalestraPatrick
Copy link
Collaborator

Hey! Do you mind taking the screenshot with the light mode and resize the window to be smaller in such a way that the important code in the script is bigger? I'll be happy to merge the PR after that 😄

@codedbypm
Copy link
Contributor Author

Hey! Do you mind taking the screenshot with the light mode and resize the window to be smaller in such a way that the important code in the script is bigger? I'll be happy to merge the PR after that 😄

Done

@BalestraPatrick
Copy link
Collaborator

We actually removed --output MyAppLogs and open MyAppLogs in #3, do you mind doing the same?

@codedbypm
Copy link
Contributor Author

I think consistency is key in documentation. IMO then also the code block above the image should be updated. What do you think?

@BalestraPatrick
Copy link
Collaborator

Yes, you're right. Let's also change that one!

@BalestraPatrick BalestraPatrick merged commit cbf503d into MobileNativeFoundation:master Jun 15, 2019
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.

2 participants