Skip to content

chore: Bump rive on flame_rive to fix CI#3907

Merged
luanpotter merged 1 commit intomainfrom
luan.rive
Apr 25, 2026
Merged

chore: Bump rive on flame_rive to fix CI#3907
luanpotter merged 1 commit intomainfrom
luan.rive

Conversation

@luanpotter
Copy link
Copy Markdown
Member

@luanpotter luanpotter commented Apr 25, 2026

Description

A new rive patch version causes an analysis issue due to re-exporting internal components.
In order to fix we have to bump the version on pubspec as these classes didn't exist before so could not be hidden.

On CI:

Dart Analyzer - Warnings
  /home/runner/work/flame/flame/packages/flame_rive/lib/flame_rive.dart:1:1 The member 'RiveSemanticsMixin' can't be exported as a part of a package's public API.
  Warning: Try using a hide clause to hide 'RiveSemanticsMixin'.
  See https://dart.dev/diagnostics/invalid_export_of_internal_element to learn more about this problem.
  
  /home/runner/work/flame/flame/packages/flame_rive/lib/flame_rive.dart:1:1 The member 'RiveSemanticsWidget' can't be exported as a part of a package's public API.
  Warning: Try using a hide clause to hide 'RiveSemanticsWidget'.
  See https://dart.dev/diagnostics/invalid_export_of_internal_element to learn more about this problem.

Checklist

  • I have followed the Contributor Guide when preparing my PR.
  • I have updated/added tests for ALL new/updated/fixed functionality.
  • I have updated/added relevant documentation in docs and added dartdoc comments with ///.
  • I have updated/added relevant examples in examples or docs.

Breaking Change?

  • Yes, this PR is a breaking change.
  • No, this PR is not a breaking change.

@luanpotter luanpotter marked this pull request as ready for review April 25, 2026 18:15
@luanpotter luanpotter requested a review from a team April 25, 2026 18:15
@luanpotter luanpotter enabled auto-merge (squash) April 25, 2026 18:27
@luanpotter luanpotter merged commit 9f7c605 into main Apr 25, 2026
10 checks passed
@luanpotter luanpotter deleted the luan.rive branch April 25, 2026 19:45
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.

2 participants