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

[v11] 3. Remove obsolete WriteAsync overloads #3234

Merged
merged 3 commits into from
Feb 24, 2023
Merged

Conversation

nirinchev
Copy link
Member

Description

Removes old WriteAsync(Action<Realm>) in favor of WriteAsync(Action).

TODO

  • Changelog entry

@nirinchev nirinchev requested a review from LaPeste February 15, 2023 22:38
@cla-bot cla-bot bot added the cla: yes label Feb 15, 2023
@nirinchev nirinchev changed the title Remove obsolete WriteAsync overloads [v11] Remove obsolete WriteAsync overloads Feb 15, 2023
@coveralls
Copy link

coveralls commented Feb 15, 2023

Pull Request Test Coverage Report for Build 4224398269

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.1%) to 81.26%

Totals Coverage Status
Change from base Build 4188765844: -0.1%
Covered Lines: 5884
Relevant Lines: 7109

💛 - Coveralls

Copy link
Contributor

@LaPeste LaPeste left a comment

Choose a reason for hiding this comment

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

I had just a minor question. Beside that curiosity, the PR looks good.

@@ -575,7 +575,8 @@ public void UsingDisposedRealm_ShouldThrowObjectDisposedException()
Assert.That(() => realm.RemoveAll<Person>(), Throws.TypeOf<ObjectDisposedException>());
Assert.That(() => realm.Write(() => { }), Throws.TypeOf<ObjectDisposedException>());

await TestHelpers.AssertThrows<ObjectDisposedException>(() => realm.WriteAsync(_ => { }));
await TestHelpers.AssertThrows<ObjectDisposedException>(() => realm.WriteAsync(() => { }));
await TestHelpers.AssertThrows<ObjectDisposedException>(() => realm.RefreshAsync());
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess this should have been here since the beginning. Correct?

@nirinchev nirinchev changed the title [v11] Remove obsolete WriteAsync overloads [v11] 3. Remove obsolete WriteAsync overloads Feb 16, 2023
@cla-bot cla-bot bot added the cla: yes label Feb 20, 2023
Base automatically changed from ni/v11-dynamic to main-v11 February 24, 2023 09:27
@nirinchev nirinchev merged commit a07621c into main-v11 Feb 24, 2023
@nirinchev nirinchev deleted the ni/v11-async-write branch February 24, 2023 13:14
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants