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

fix: iOS13 Swift Sample is not compiling #1935

Merged
merged 1 commit into from
Jul 5, 2022

Conversation

brustolin
Copy link
Contributor

iOS13 sample shares files with iOS Swift Sample. Sometimes we add files to one without adding it to the other preventing it from compiling.

#skip-changelog

Copy link
Member

@philipphofmann philipphofmann left a comment

Choose a reason for hiding this comment

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

@brustolin, can we compile this in CI to avoid the same problem in the future, please? I think the build workflow would be the correct place.

@brustolin
Copy link
Contributor Author

@brustolin, can we compile this in CI to avoid the same problem in the future, please? I think the build workflow would be the correct place.

Agreed. This is probably a good task for me to learn how to create tasks for CI to run.

Copy link
Member

@philipphofmann philipphofmann left a comment

Choose a reason for hiding this comment

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

LGTM

@philipphofmann philipphofmann merged commit 84efc48 into master Jul 5, 2022
@philipphofmann philipphofmann deleted the fix/ios13-sample-fix branch July 5, 2022 06:21
kevinrenskers added a commit that referenced this pull request Jul 5, 2022
* master:
  build(deps): bump fastlane from 2.206.2 to 2.207.0 (#1937)
  fixing sample not compiling (#1935)
  style: Configure the root of the project to use four spaces for indentation (#1934)
  meta: Add LOGAF scale to Contributing (#1933)
  meta: Mention draft PRs in Contributing (#1932)
  meta: Add sut to contributing (#1931)
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

2 participants