Skip to content

[chore]: bump minimum deployment & swift versions #186

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

Merged
merged 11 commits into from
Feb 3, 2023

Conversation

jamieQ
Copy link
Contributor

@jamieQ jamieQ commented Jan 31, 2023

Issue

  • we'd like to bump the minimum deployment target and swift version to facilitate development going forward

Description

  • bump minimum iOS deployment target to 14.0
  • bump swift version to 5.7
  • remove an unneeded config in our gemfile
  • bump xcode version in CI configs to 14.2.0
  • bump simulator target in CI configs to 16.2
  • add ENABLE_TESTING_SEARCH_PATHS to WorkflowTesting podspec to resolve search path issues induced by deployment target change
  • remove VERSION file in favor of version.rb, and standardize version references

note: this change will require a major version bump for the next release that includes it

Checklist

  • Unit Tests
  • UI Tests
  • Snapshot Tests (iOS only)
  • I have made corresponding changes to the documentation

@jamieQ jamieQ marked this pull request as ready for review February 1, 2023 20:51
@jamieQ jamieQ requested a review from a team as a code owner February 1, 2023 20:51
@mjohnson12
Copy link
Collaborator

You could probably follow this up with removing the @Availability on the iOS 13+ code.

Copy link
Contributor

@lickel lickel left a comment

Choose a reason for hiding this comment

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

note: this change will require a major version bump for the next release that includes it

Would you re-adjust things such that concurrency's version == workflow again?

Also, have you considered throwing more stuff in version.rb for maintenance? Eg the target OSes/swift version

@jamieQ
Copy link
Contributor Author

jamieQ commented Feb 2, 2023

Would you re-adjust things such that concurrency's version == workflow again?

yep! that's the plan anyway. also have some additional primary associated types to add

Also, have you considered throwing more stuff in version.rb for maintenance? Eg the target OSes/swift version

that's a good suggestion – i'll look into it!

@lickel
Copy link
Contributor

lickel commented Feb 2, 2023

i'll look into it!

Market uses that pattern if you want inspiration

@jamieQ jamieQ force-pushed the jquadri/bump-deploy-swift-versions branch from 62d94ba to 01e4289 Compare February 3, 2023 10:43
s.swift_versions = ['5.0']
s.ios.deployment_target = '11.0'
s.swift_versions = [WORKFLOW_SWIFT_VERSION]
s.ios.deployment_target = WORKFLOW_IOS_DEPLOYMENT_TARGET
Copy link
Collaborator

Choose a reason for hiding this comment

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

I love this change instead of repeating the value everywhere 👍

@jamieQ jamieQ merged commit 8d34d05 into main Feb 3, 2023
@jamieQ jamieQ deleted the jquadri/bump-deploy-swift-versions branch February 3, 2023 20:48
jamieQ added a commit that referenced this pull request Feb 6, 2023
* main:
  [fix]: use weak reference to internal sinks when vending to clients (#189)
  [feat]: add primary associated types to more protocols (#188)
  [chore]: pre major version bump cleanup (#187)
  [chore]: bump minimum deployment & swift versions (#186)
  Abstract ViewEnvironment to shared framework (#185)
  Bump activesupport from 6.1.4.4 to 6.1.7.1 (#183)
  [feat]: add primary associated types to `Workflow` protocol (#181)
  [chore]: update swiftformat ifdef indent rule to no-indent (#182)
jamieQ added a commit that referenced this pull request Feb 25, 2023
…ow-workflow-conformance

* origin/main:
  Expose AnyScreen.wrappedScreen for inspection (#193)
  Improve introspection for SignalProducerWorkflow actions (#192)
  [release]: bump version to 2.2.0 & remove separate concurrency version (#191)
  [feat]: add runtime observation API (#168)
  [chore]: refactor some internal actions to use existential any (#190)
  [fix]: use weak reference to internal sinks when vending to clients (#189)
  [feat]: add primary associated types to more protocols (#188)
  [chore]: pre major version bump cleanup (#187)
  [chore]: bump minimum deployment & swift versions (#186)
  Abstract ViewEnvironment to shared framework (#185)
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