Skip to content

Conversation

@CarlGao4
Copy link

@CarlGao4 CarlGao4 commented Apr 19, 2025

close #2
close #14

@Xmader

Jojo-Schmitz and others added 30 commits April 9, 2024 17:32
To avoid messages like this:

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: xxxx
For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
`sudo` does not preserve environment variables, while we need the `LUPDATE_ARGS` environment variable. Of course there are ways around that, but the use of sudo is completely unnecessary here, so let's simply remove it.
…entals

Fix musescore#22229:  export ornament accidentals to MusicXML
reg.: enumeration value not handled in switch [-Wswitch]
reg.: will be initialized after [-Wreorder]
reg.: this statement may fall through [-Wimplicit-fallthrough=]
reg.: variable set but not used [-Wunused-but-set-variable]
reg.: comparison of unsigned expression in ‘>= 0’ is always true [-Wtype-limits]
reg.: suggest parentheses around ‘&&’ within ‘||’ [-Wparentheses]
reg.: type qualifiers ignored on function return type [-Wignored-qualifiers]
reg.: comparison of integer expressions of different signedness: ‘int’ and ‘size_t’ {aka ‘long unsigned int’} [-Wsign-compare]
Error looks like:
/home/runner/work/_temp/55608d39-c27e-4b05-bdb8-6c3a00820a6f.sh: line 43: [: !=: unary operator expected
This way, VST plugins that also use qml won't have access to our internal qml folder
…hythms_crash-430

Fix musescore#21953: Crash on regroup rhythms in some scores (4.3.0)
Fix empty figured-bass in MusicXML export
@CarlGao4 CarlGao4 marked this pull request as draft April 20, 2025 17:23
@CarlGao4 CarlGao4 force-pushed the webmscore branch 2 times, most recently from bf6c684 to 815f2d2 Compare April 21, 2025 08:40
@CarlGao4 CarlGao4 marked this pull request as ready for review April 21, 2025 08:41
@CarlGao4
Copy link
Author

Currently, it still only support emsdk<2.0.24 and node.js<18

@CarlGao4
Copy link
Author

I'm not supporting Node.js 18 in this PR

Reason: #10 (comment)

@PeterNjeim
Copy link
Member

Is this ready to be merged or are more commits needed

@PeterNjeim
Copy link
Member

I just realized I don't have maintainer permissions on this repository, so I can't merge it

@CarlGao4
Copy link
Author

CarlGao4 commented Apr 21, 2025

It's ready now, the CI has passed: https://github.com/CarlGao4/webmscore/actions/runs/14575709935
I just say that currently #10 could not be solved unless we upgrade Qt to 6.7

@CarlGao4
Copy link
Author

@Xmader

@CarlGao4
Copy link
Author

CarlGao4 commented Oct 7, 2025

I'll close this PR and maintain my new repo. Maybe I can upload the project to npmjs using a new name like "webmscore4"

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.

Conversion doesn't seem to work anymore for recent MuseScore 4 files Unmapped drum note *number*