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 patch version in unpkg url #94

Merged
merged 1 commit into from
Nov 30, 2023
Merged

Fix patch version in unpkg url #94

merged 1 commit into from
Nov 30, 2023

Conversation

gudzpoz
Copy link
Contributor

@gudzpoz gudzpoz commented Oct 31, 2023

Try to specify optional patch version in the unpkg url, removing any trailing labels (like -SNAPSHOT or whatsoever).

fix #93

@ceifa
Copy link
Owner

ceifa commented Nov 5, 2023

Why the regex is needed? The fix wouldnt be just put the raw version variable?

@gudzpoz
Copy link
Contributor Author

gudzpoz commented Nov 12, 2023

I was wondering if the version string would include some extra suffixes as is specified by semver:

<valid semver> ::= ...
                 | <version core> "-" <pre-release> "+" <build>

But I guess the regex approach is still problematic since vernum1-SNAPSHOT will then rely on a published vernum1 version... Anyway, I've updated to simply use the raw version string. Thank you!

@ceifa
Copy link
Owner

ceifa commented Nov 23, 2023

Build is broken because of a bug on emscripten: emscripten-core/emscripten#20726

@ceifa
Copy link
Owner

ceifa commented Nov 30, 2023

Merging it because the main is fixed and should build just great

@ceifa ceifa merged commit 730e21e into ceifa:main Nov 30, 2023
0 of 3 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.

[bug] New patch release might break older versions: [email protected] breaking [email protected]
2 participants