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

chore: fix build issues and Preview Content #1234

Merged
merged 1 commit into from
Aug 24, 2022
Merged

Conversation

krodak
Copy link
Contributor

@krodak krodak commented Aug 24, 2022

Purpose

This PR fixes issues with SwiftUI "Preview Content" mode that enables easy UI prototyping and also some build issues that started happening after changing the branch or multiple builds, after changes introduced by: #1229

Scope

  • add checks for all Build Phases that potentially modify files during indexing
  • add checks for *.udl file processing Build Rules to avoid file generation during indexing or Preview Content

TODOs

Discussion

This is XCode 13+ related issue, found solution while googling this out on one of the issues within Cuckoo Github page (tool that autogenerates files for Swift)

Screenshots

Properly not generating files when indexing

CleanShot 2022-08-24 at 13 26 59@2x

Previews working

CleanShot 2022-08-24 at 10 46 04@2x

@krodak krodak added the ios label Aug 24, 2022
@krodak krodak self-assigned this Aug 24, 2022
@montekki montekki merged commit c1227db into master Aug 24, 2022
@montekki montekki deleted the fix-preview-mode branch August 24, 2022 12:24
krodak added a commit that referenced this pull request Aug 24, 2022
* master:
  chore: add derived_keys_count to SeedNameCard (#1235)
  ci: pin gha versions (#1232)
  chore: fix build issues and Preview Content (#1234)
  chore: Revert "chore: fix build issues and Preview Content" (#1233)
  chore: fix build issues and Preview Content
  chore: scale identicons (#1230)
  chore: add integration tests to generate_message (#1219)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants