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

Misleading version text on stimulus dist files #714

Open
tonywok opened this issue Aug 14, 2023 · 3 comments
Open

Misleading version text on stimulus dist files #714

tonywok opened this issue Aug 14, 2023 · 3 comments

Comments

@tonywok
Copy link

tonywok commented Aug 14, 2023

Hi there,

First of all thank you for all your hard work. I've been a happy stimulus user for some time now.

Noticed something (maybe?) weird when investigating a bug that was resolved in #648. I wanted to be sure that I had the fix since I was still seeing the pre-fix behavior 🤔

Anyway, I was looking at the 3.2.2 tag and noticed that if you download the files from https://registry.yarnpkg.com/@hotwired/stimulus/-/stimulus-3.2.2.tgz you seem to get 3.2.1 😅.

Pulling this thread a bit more, it seems https://www.npmjs.com/package/@hotwired/stimulus/v/3.2.2?activeTab=code is also displaying 3.2.1.

edit: apologies, it seems npm's website doesn't link the ui state (irony), so you have to manually navigate to the file. Including screenshot for now

image

Anyway, this is making me wonder if 3.2.2 was in fact released. Apologies in advance if this ends up being an exercise in rationalization. Trying to convince myself otherwise.


Update: See #714 (comment) for explanation.

@marcoroth
Copy link
Member

Hey @tonywok, yeah Stimulus 3.2.2 is fully released and is actually the version which is up on NPM.

Sadly the comment at the top of the dist file is misleading. Depending on how you run the yarn/npm release cycle you build the library first and then bump the version number as part of the release process. This can lead to such weird artifacts as you are describing here. Like, at the point when the library was built the "current version" was still the "previous version" and after that the version gets bumped.

Either way, we should probably make sure that this doesn't happen anymore or remove the version number from the dist-files all together to avoid confusion.

@tonywok
Copy link
Author

tonywok commented Aug 14, 2023

Ah, I see. Thank you for the prompt response. I'll go ahead and update the issue title to reflect what's actually occurring.

@tonywok tonywok changed the title Is 3.2.2 fully released? Misleading version text on stimulus dist files Aug 14, 2023
@marcoroth
Copy link
Member

marcoroth commented Aug 14, 2023

If you look at the package.json inside the 3.2.2 version on npm you can in fact see it's the right version. Sorry for the confusion.

But this definitely something we need to sort out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants