Update Bravura to 1.481 - #34489
Conversation
📝 WalkthroughWalkthroughUpdated Bravura and Bravura Text release history through versions 1.480 and 1.481. Refreshed licensing, SMuFL references, repository links, support guidance, and Unicode-input instructions. Updated Bravura Text documentation to version 1.2 dated 6 August 2026. Changed the bundled font resource mapping to use 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning Review ran into problems🔥 ProblemsLinked repositories: Public OSS repositories can only analyze public repositories installed in this organization. No linked repositories were analyzed; skipped Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@fonts/bravura/FONTLOG.txt`:
- Line 486: Update the changelog entry for noteheadDiamondDoubleWhole and
noteheadDiamondWhole to use the past-tense “Improved” instead of “Improve,”
matching the surrounding completed-change entries.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: 413437be-0efc-44da-af7f-70b456963dd4
⛔ Files ignored due to path filters (2)
fonts/bravura/Bravura.otfis excluded by!**/*.otffonts/bravura/BravuraText.otfis excluded by!**/*.otf
📒 Files selected for processing (4)
fonts/bravura/FONTLOG.txtfonts/bravura/LICENSE.txtfonts/bravura/README.mdfonts/bravura/bravura_metadata.json
|
Note GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer. |
|
I guess we can go to 1.481 now, which got released a couple minutes ago. Or just wait a little longer, util the dust settles, seems Daniel Spreadbury is heavily working on Bravura ATM |
|
Note GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer. |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@fonts/bravura/bravura-text.md`:
- Line 57: Update the navigation terminology in the paragraph: use the current
macOS path “System Settings → Keyboard → Keyboard Shortcuts → Input Sources”
instead of the legacy pane/tab wording, and replace “OS X” with “macOS”;
preserve the shortcut instructions and examples.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: 56eb6384-2139-445b-ae39-751cd359879c
⛔ Files ignored due to path filters (2)
fonts/bravura/Bravura.otfis excluded by!**/*.otffonts/bravura/BravuraText.otfis excluded by!**/*.otf
📒 Files selected for processing (7)
fonts/bravura/Bravura.jsonfonts/bravura/FONTLOG.txtfonts/bravura/LICENSE.txtfonts/bravura/README.mdfonts/bravura/bravura-text.mdfonts/bravura/bravura_metadata.jsonsrc/engraving/data/fonts/fonts_Bravura.qrc
🚧 Files skipped from review as they are similar to previous changes (3)
- fonts/bravura/README.md
- fonts/bravura/LICENSE.txt
- src/engraving/data/fonts/fonts_Bravura.qrc




which got released just today (7Aug26)