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

Not updating version when adding features #386

Closed
OmgImAlexis opened this issue Mar 8, 2017 · 12 comments
Closed

Not updating version when adding features #386

OmgImAlexis opened this issue Mar 8, 2017 · 12 comments

Comments

@OmgImAlexis
Copy link

When features get added to this repo the minor version should be bumped and the latest version should be pushed to npm. Right now I have version 2.8.1 installed from npm which is the latest yet the version I have installed is different to what's in this repo with the exact same version number without even a minor version bump.

From what I can see about 25 commits have been added since the npm update. Maybe you should move to using a dev and master branch and only update npm when dev is merged into master like most projects?

@OmgImAlexis
Copy link
Author

Closes: #384

@posva
Copy link
Member

posva commented Mar 8, 2017

Yes, we can do that but I really don't understand what's the matter here. It's normal for the repository code to be more advanced than the released version after all. If you want the latests version you don't rely on the master branch, you use a dist-tag like latest, and if you want the dev version you usually use the dev branch, which in this case is master.

@OmgImAlexis
Copy link
Author

It's actually normal to use a dev branch as master should be the MASTER copy which is the latest public version. This is exactly what I've seen on every other large project even ones I work on.

@posva
Copy link
Member

posva commented Mar 8, 2017

It's not about being normal or not, it's about what problem are you facing that can be solved by changing the branches names. The reason I'm asking this is to understand how important the task is to prioritise it

@OmgImAlexis
Copy link
Author

Well going to the repo and new things being added that don't work will just end up causing issues like mine where I expected the complete field to work yet it doesn't as it's not up to date on npm as master is for some reason out of sync with it.

@posva
Copy link
Member

posva commented Mar 8, 2017

If you are using the dev version, you're doing it at your own risk. I still don't see the matter

@OmgImAlexis
Copy link
Author

If there's something added to the master then the MASTER on npm which is the latest version should also have the thing added hence the issue.

If you don't understand it maybe leave this issue for someone else on the development team to deal with.

@posva
Copy link
Member

posva commented Mar 8, 2017

You're still not answering my question about the issue you had. I know what is a git flow, lol... Whatever...
I'll create a new proper issue but just so you know, you don't ned a dev/master flow for every project (large or not)

If you don't understand it maybe leave this issue for someone else on the development team to deal with.

lol. Next time, keep that for yourself 😉

@OmgImAlexis
Copy link
Author

The issue is you've got up to date code on your repo which is leading people to think they're implemented in the version that can be downloaded from npm.

This is not the case.

@OmgImAlexis
Copy link
Author

The other issue is the versions isn't getting bumped when it should. 2.8.1 for example is listed on both the version of this repo with and without a complete field. That's an issue.

@posva
Copy link
Member

posva commented Mar 8, 2017

Finally! Thanks

@luxueyan
Copy link

Yes! I agree with the @OmgImAlexis .For example, the complete callback function is not supported in the version that can be downloaded from npm !

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

No branches or pull requests

3 participants