Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jan 20, 2022. It is now read-only.

Conversation

@jnd71
Copy link
Contributor

@jnd71 jnd71 commented Feb 27, 2014

I'm working through issue 212, and realized I had changed the gitignore file to use my Vagrant workflow. I wasn't sure if y'all wanted such a trivial change as its own PR, a separate commit in a PR, or buried in another commit since this change is side effect. The contributing lists "small simple changes" per commit, so I thought I'd just open a PR and ask just to be safe.

@ianoc
Copy link
Collaborator

ianoc commented Feb 27, 2014

Thanks!, Smaller PR's are the best ones.

ianoc added a commit that referenced this pull request Feb 27, 2014
Vagrant folder added to .gitignore
@ianoc ianoc merged commit c8514fc into twitter:develop Feb 27, 2014
@jnd71 jnd71 deleted the vagrant branch February 27, 2014 19:26
snoble pushed a commit to snoble/summingbird that referenced this pull request Sep 8, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants