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

Ignore bin directory #5

Closed
wants to merge 1 commit into from
Closed

Conversation

karlfreeman
Copy link

- used by bundler's --binstubs
@karlfreeman karlfreeman deleted the ignore-bin branch April 10, 2014 10:40
@karlfreeman
Copy link
Author

Going to sweep across all repo's with changes bundled up rather than individual ones.

@fuadsaud
Copy link

Binstubs should be commited, right?

@jodosha
Copy link
Member

jodosha commented Apr 16, 2014

@fuadsaud Nope.

@fuadsaud
Copy link

Hum, didn't know about that.

@jodosha
Copy link
Member

jodosha commented Apr 17, 2014

@fuadsaud Binstubs are a facility while developing applications, not gems.

@fuadsaud
Copy link

I understand what they are, but I always assumed it was a good idea to commit them (at least when building apps).

@jwreagor
Copy link

@fuadsaud it's not common practice to commit binstubs since they are a developer construct and will indefinitely change from developer to developer.

I work on Ruby projects with people using all kinds of different VM managers and I've found that to keep things clean and simple I generate my binstubs within .bin and add ./.bin to my shell's PATH. This helps when working on libraries that do and do not have a commited /bin. You can also place .bin in your global ~/.gitignore and never worry about it again.

timriley added a commit that referenced this pull request Mar 15, 2020
Fix error: Template file is not valid US-ASCII
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

4 participants