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

Bump changes to 0.51.0 Changes Version #109

Merged
merged 5 commits into from
Jan 3, 2024

Conversation

@dikasetiadi dikasetiadi added the enhancement New feature or request label Jan 3, 2024
@dikasetiadi dikasetiadi self-assigned this Jan 3, 2024
@dikasetiadi dikasetiadi changed the base branch from main to feature/rxtca-0.51.0 January 3, 2024 05:20
Comment on lines +10 to +18
// MARK: - Deprecated after 0.50.4

@available(
*,
deprecated,
message: "Use 'Effect<Action>.Send' instead."
)
public typealias Send<Action> = Effect<Action>.Send

Copy link
Contributor

Choose a reason for hiding this comment

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

please try to remove this (I don't think we have any usage of send yet)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this one is from pointfreeco/swift-composable-architecture#1911
it was also used internally by some core like store etc. so this one for also the warning if the codes use Send (if we pulling another core codes later on, we can see warning and change it into new one) wdyt gan? @jeffersonsetiawan

Copy link
Contributor

Choose a reason for hiding this comment

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

If we remove this code and there is no error, I think we can go to remove it dik, as Toko is not using this Send code yet CMIIW

@jeffersonsetiawan jeffersonsetiawan merged commit c8d3a81 into feature/rxtca-0.51.0 Jan 3, 2024
7 checks passed
@jeffersonsetiawan jeffersonsetiawan deleted the feature/bump-to-0.51.0 branch January 3, 2024 06:37
jeffersonsetiawan pushed a commit that referenced this pull request Jan 3, 2024
* empty commit

* Bump changes to 0.50.1 Changes Version (#106)

* update to changes PR #1879

* add changes from #1882

* fix build on xcode 13.4.1

* Bump changes to 0.50.2 Changes Version (#107)

* apply changes from #1883

* apply changes from #1890

* apply changes from #1893

* apply changes from #1897

* Bump changes to 0.50.3 Changes Version (#108)

* update package.swift & missmatch version podspec

* apply partial changes from #1907

* Bump changes to 0.51.0 Changes Version (#109)

* apply partial changes from #1917

* fix test on RxTca

* apply changes from #1911

* apply changes from #1920

* apply changes from #1900
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants