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 #138: [jaspr_riverpod] update vsyncOverride to flutterVsyncs #139

Merged
merged 2 commits into from
Dec 14, 2023

Conversation

gadfly361
Copy link
Contributor

@gadfly361 gadfly361 commented Nov 14, 2023

Description

This PR is a fix for #138.

In riverpod,vsyncOverride was removed in riverpod v2.4.6 (rrousselGit/riverpod#3072) and replaced with flutterVsyncs. This PR makes the same replacement in jaspr_riverpod.

Type of Change

  • ❌ Breaking change
  • ✨ New feature or improvement
  • 🛠️ Bug fix
  • 🧹 Code refactor
  • 📝 Documentation
  • 🗑️ Chore

Ready Checklist

  • I've read the Contribution Guide.
  • In case this PR changes one of the core packages, I've modified the respective CHANGELOG.md file using
    the semantic_changelog format.
  • I updated/added relevant documentation (doc comments with ///).
  • I added myself to the AUTHORS file (optional, if you want to).

If you need help, consider asking for advice on the #contribute channel on Discord.

@gadfly361 gadfly361 requested a review from schultek as a code owner November 14, 2023 00:15
Copy link

github-actions bot commented Nov 14, 2023

Package Version Report

The following packages have been updated:
jaspr_riverpod : 0.3.7 -> 0.3.8

@gadfly361 gadfly361 force-pushed the fix/issue-138/vsync-override branch 2 times, most recently from 7874fa2 to 92b0af8 Compare November 14, 2023 17:12
@gadfly361 gadfly361 force-pushed the fix/issue-138/vsync-override branch from 92b0af8 to b9a437c Compare November 14, 2023 17:18
@gadfly361 gadfly361 changed the title fix #138: [jaspr_riverpod] add missing variable to vsync override fix #138: [jaspr_riverpod] update vsyncOverride to flutterVsyncs Nov 14, 2023
packages/jaspr_riverpod/CHANGELOG.md Outdated Show resolved Hide resolved
@gadfly361 gadfly361 requested a review from schultek November 14, 2023 17:49
@Pawelo93
Copy link

Pawelo93 commented Dec 13, 2023

Hey, is there any issue with this PR? Or You can merge this 🙏

@schultek schultek merged commit 01dd6db into schultek:main Dec 14, 2023
5 of 6 checks passed
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.

3 participants