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

Vinxi 0.2.1 -> 0.3.3 (Note: this changes default from vite 4 -> 5) #1331

Merged
merged 1 commit into from
Feb 20, 2024

Conversation

birkskyum
Copy link
Contributor

@birkskyum birkskyum commented Feb 14, 2024

Related to

I've found usage of vite/runtime (link below), so I don't trust vinxi 0.3.x, at least the vinxi run to be compatible with vite 4:

nksaraf/vinxi@576440c#diff-1f1d67ddefb4f54313fdeafa8e44b64ad7b264539430e97790d1cd4414756581L401

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tests for the changes have been added (for bug fixes / features)

I went through all the examples to check it didn't break anything. This manual method flagged an issue with vinxi 0.3.0, which is resolved by 0.3.1.

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • infrastructure changes
  • Other... Please describe:

What is the current behavior?

Vinxi 0.2.1 (vite 4 default)

What is the new behavior?

Vinxi 0.3.3 (vite 5 default)

Other information

I set the changeset to a minor, even though I'd consider it a major, but I don't know how changeset handles 0.x.x releases, so let me know if it should be adjusted. Even though I didn't detect anything breaking, it might be reasonable to bump to v0.6.0.

Copy link

changeset-bot bot commented Feb 14, 2024

🦋 Changeset detected

Latest commit: dbc0c68

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@solidjs/start Minor
example-notes Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@birkskyum birkskyum changed the title Vinxi 0.2.1 -> 0.3.1 (thus change default from vite 4 -> 5) Vinxi 0.2.1 -> 0.3.x (thus change default from vite 4 -> 5) Feb 14, 2024
@birkskyum birkskyum changed the title Vinxi 0.2.1 -> 0.3.x (thus change default from vite 4 -> 5) Vinxi 0.2.1 -> 0.3.3 (thus change default from vite 4 -> 5) Feb 14, 2024
@birkskyum birkskyum changed the title Vinxi 0.2.1 -> 0.3.3 (thus change default from vite 4 -> 5) Vinxi 0.2.1 -> 0.3.3 (Note: this changes default from vite 4 -> 5) Feb 14, 2024
@birkskyum birkskyum force-pushed the vinxi-0.3.1 branch 3 times, most recently from a863466 to 2a203a9 Compare February 15, 2024 09:11
@ryansolid ryansolid changed the base branch from main to next February 20, 2024 22:29
@ryansolid ryansolid merged commit b3f4af1 into solidjs:next Feb 20, 2024
1 check 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.

None yet

2 participants