Skip to content
This repository has been archived by the owner on Mar 18, 2021. It is now read-only.

Code for NPM 1.2.0 version does not match taggeed version in github #226

Closed
austinnichols101 opened this issue Jan 9, 2017 · 4 comments
Closed
Labels

Comments

@austinnichols101
Copy link

It appears that v1.2.0 from npm has bundled code from another version (I think 1.1). For example, the npm version specifies "react-router": "2.7.0" as a dependency.

npm install react-router-breadcrumbs@1.2.0

**cat node_modules/react-router-breadcrumbs/package.json | grep 1.2.0**
        "raw": "react-router-breadcrumbs@1.2.0",
        "rawSpec": "1.2.0",
        "spec": "1.2.0",
  "_from": "react-router-breadcrumbs@1.2.0",
  "_id": "react-router-breadcrumbs@1.2.0",
    "tmp": "tmp/react-router-breadcrumbs-1.2.0.tgz_1474404409315_0.8684582761488855"
    "raw": "react-router-breadcrumbs@1.2.0",
    "rawSpec": "1.2.0",
    "spec": "1.2.0",
  "_resolved": "https://registry.npmjs.org/react-router-breadcrumbs/-/react-router-breadcrumbs-1.2.0.tgz",
  "_spec": "react-router-breadcrumbs@1.2.0",
    "tarball": "https://registry.npmjs.org/react-router-breadcrumbs/-/react-router-breadcrumbs-1.2.0.tgz"
  "version": "1.2.0"
**cat node_modules/react-router-breadcrumbs/package.json | grep '"react-router"'**
    "react-router": "2.7.0",
    "react-router",
    "react-router": "^2.0.1"
@nutgaard
Copy link
Owner

nutgaard commented Jan 9, 2017

Hi, and thanks for posting the issue.

The difference between 1.1 and 1.2 are were a simple bugfix, hence the similarities in package.json. I've inspected the zip-files, and from what I can see the changes are included in 1.2.

The library does not bundle react-router as a dependency at all, but it is listed as a devDependency and peerDependency.

There has however been some changes since 1.2 that are not released yet, most prominent is allowing react-router@^3.0.0 as a peerDependency. I'll try to get that one out as soon as possible.

I will be closing this issue for now, but if I've misunderstood anything please let me know so I can correct any potential bugs. :)

@nutgaard nutgaard closed this as completed Jan 9, 2017
@austinnichols101
Copy link
Author

austinnichols101 commented Jan 9, 2017

My mistake - I was mis-reading devDependencies for dependencies. However, I'm showing a peer dependency of ^2.0.1 here which is keeping me from running react-router 3.0.0. Eagerly awaiting a release supporting 3.0.0.

b70b6a1

@nutgaard
Copy link
Owner

Yeah, I discovered that when going through the commit-history yesterday. Seems like a slip-up on my behalf, and will try to get a new version out today.

Anyways, I'm reopening this issue to address the issue.

@nutgaard nutgaard reopened this Jan 10, 2017
@nutgaard nutgaard added the bug label Jan 10, 2017
@nutgaard
Copy link
Owner

react-router-breadcrumbs@1.3.0 released which allows react-router@^3.0.0 as peerDependency. Closing this issue

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

No branches or pull requests

2 participants