Remove unnecessary joinClasses dependency

[heydiplo]

...which also ruins builds with React from CDN.

[taion]

This is fine in principle, except lib/ contains the transpiled sources. You'll want to edit the file in src/.

[heydiplo]

😁 sorry, fixed.

[taion]

Sorry, one last thing - don't update the files in lib/. We'll handle it when we release.

@mtscout6 What do you think of .gitignoreing the lib/ path and putting npm run build in the prepublish script?

LGTM otherwise, need another @react-bootstrap/collaborators reviewer though.

[mathieumg]

What do you think of .gitignoreing the lib/ path and putting npm run build in the prepublish script?

+1 👍 It's making this one-line review a pain, for instance.

LGTM.

[aabenoja]

What do you think of .gitignoreing the lib/ path and putting npm run build in the prepublish script?

👍

I would prefer to have the lib changes stripped before we pull this in, but that's just me. Looks good otherwise.

[taion]

@aabenoja I agree, I'll make a PR once we merge this.

[mathieumg]

I agree, I'll make a PR once we merge this.

Isn't that contradictory? 😛

[heydiplo]

Removed lib changes, guys, it's pretty clean now.

[mtscout6]

@taion Once we setup a separate repo for the bower release and put scripts in place to automate that then I'd be fine with removing the lib folder from the repo. The only reason it's committed now is for the bower release.

[mtscout6]

See issue #48

[taion]

Brain fart, forgot that Bower doesn't work like NPM and actually looks at the repo.