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

2.3.1 Update #306

Closed
wants to merge 2 commits into from
Closed

2.3.1 Update #306

wants to merge 2 commits into from

Conversation

dallas
Copy link

@dallas dallas commented Mar 4, 2013

Copied commits directly from https://github.com/twitter/bootstrap/commits/master in regards to the 2.3.1 release

@dallas
Copy link
Author

dallas commented Mar 4, 2013

Sorry, didn't mean to make a separate issue—it's been a while since I've done pull requests. I can't remember how to tie a pull request to an already open issue. 😦

Original issue: #305. My bad!

@trisweb
Copy link
Contributor

trisweb commented Mar 4, 2013

Cool - thanks! It's entirely fine to have an issue and a pull request to fix it. Normal, in fact. No worries.

@trisweb
Copy link
Contributor

trisweb commented Mar 4, 2013

If you want to be really nice, you can also add the changes to README.md and other files for the bootstrap-sass version change - it would be 2.3.1.0

@dallas
Copy link
Author

dallas commented Mar 4, 2013

Alright, I'll give it a shot!

@dallas
Copy link
Author

dallas commented Mar 4, 2013

Oh dang, it just shows up like magic! That's great 😄

@trisweb
Copy link
Contributor

trisweb commented Mar 4, 2013

Indeed it does! This github thing is pretty neat.

@trisweb
Copy link
Contributor

trisweb commented Mar 4, 2013

@thomas-mcdonald take a look and merge/release if it looks good.

Also @dallas -- are we absolutely certain these are the only changes in 2.3.1? No CSS changes at all? Thanks!

@dallas
Copy link
Author

dallas commented Mar 4, 2013

Pretty sure. I just walked through the commit history for the actual twitter/bootstrap repository since their release of 2.3.0 on Feb. 7.

@dallas
Copy link
Author

dallas commented Mar 4, 2013

You can review all those commits using this compare view: twbs/bootstrap@v2.3.0...v2.3.1

@trisweb
Copy link
Contributor

trisweb commented Mar 5, 2013

Excellent, looks good.

:shipit:

@m5o
Copy link

m5o commented Mar 7, 2013

sorry for the delay - now we are also ready -- if someone will compare.

jlong/sass-bootstrap-defunct#56

anyway, the changes made by @dallas looks good 👍

@trisweb
Copy link
Contributor

trisweb commented Mar 7, 2013

Thanks! Just to be safe, I'll run my automated updater with another branch,
diff, and make sure everything matches up.

trisweb added a commit to trisweb/bootstrap-sass that referenced this pull request Mar 12, 2013
@trisweb
Copy link
Contributor

trisweb commented Mar 12, 2013

Looks good! All changes match up. Closing in favor of the automatically merged changes, but thank you again for your pull.

@trisweb trisweb closed this Mar 12, 2013
rroblak pushed a commit to insight-meditation-center/imc-bootstrap-sass that referenced this pull request Nov 9, 2015
bluedone added a commit to bluedone/bootstrap-rubygem that referenced this pull request Jun 6, 2023
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.

None yet

3 participants