Skip to content

Conversation

maurizi
Copy link
Contributor

@maurizi maurizi commented Dec 9, 2014

Reverts #1844. Opening this as a separate PR to avoid any regressions given the imminent release.

@RickMohr
Copy link
Contributor

RickMohr commented Dec 9, 2014

What was the old version? Do we need to worry about breaking changes?

@maurizi
Copy link
Contributor Author

maurizi commented Dec 9, 2014

@RickMohr The old version was 3.1.1
If they're really following semantic versioning there should be no breaking changes.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling e74470d on maurizi:update-bootstrap into 295a869 on OpenTreeMap:master.

@RickMohr
Copy link
Contributor

RickMohr commented Dec 9, 2014

Well, who knows about that "if"? Finding any introduced regressions in OTM would be tough. Can we (or a designer) research the safety?

@maurizi
Copy link
Contributor Author

maurizi commented Dec 9, 2014

Well they have this in their README.

I skimmed https://github.com/twbs/bootstrap/releases for versions 3.2.0-3.3.1 and didn't see any breaking changes. Looks like it's mostly bug fixes and and tiny new features, with a couple of deprecated (but not removed) features.

@maurizi
Copy link
Contributor Author

maurizi commented Dec 9, 2014

Though I still have no intention of merging this until after we start our release process.

@RickMohr
Copy link
Contributor

RickMohr commented Dec 9, 2014

Thanks, +1

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling e2801e8 on maurizi:update-bootstrap into 295a869 on OpenTreeMap:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 7755321 on maurizi:update-bootstrap into 295a869 on OpenTreeMap:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) when pulling 82bed0c on maurizi:update-bootstrap into 18fb890 on OpenTreeMap:master.

maurizi added a commit that referenced this pull request Jan 19, 2015
@maurizi maurizi merged commit c36ed48 into OpenTreeMap:master Jan 19, 2015
@maurizi maurizi deleted the update-bootstrap branch January 19, 2015 16:40
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.

3 participants