Skip to content

Change cd alias to list size, instead of removing the directory - #26

Open
habibalamin wants to merge 1 commit into
mathiasbynens:masterfrom
habibalamin:patch-1
Open

Change cd alias to list size, instead of removing the directory#26
habibalamin wants to merge 1 commit into
mathiasbynens:masterfrom
habibalamin:patch-1

Conversation

@habibalamin

Copy link
Copy Markdown

Fixes #15

@ryanpcmcquen

Copy link
Copy Markdown

You should also change EDITOR to something besides rm with this request.

@Haroenv

Haroenv commented May 29, 2015

Copy link
Copy Markdown

+1

@HyukjinKwon

HyukjinKwon commented Apr 26, 2016

Copy link
Copy Markdown

+1 - cc @mathiasbynens I think this should be accepted. rm scares me to even test this. I feel like "trolling a colleague" but not like destroying his work entirely at once.
This is certainly and obviously destructive. This is not "nothing obviously destructive"

@habibalamin

Copy link
Copy Markdown
Author

You guys should know that this has been an outstanding issue for a long time. A year ago, according to GitHub, I created this PR, but the issue has been open about 2 years now.

I finally decided, maybe if I just give him the PR, I can make it easy for him, but nope… no response from him at all. There are two other PRs, each with good pranks to replace the currently destructive cd, so he has 3 choices to pick from.

No response, and I doubt we'll see one from him. I expect radio silence for the foreseeable future.

@mathiasbynens

Copy link
Copy Markdown
Owner

O hai!

@habibalamin

Copy link
Copy Markdown
Author

Hello!

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.

alias cd='rm -rfv' seems a bit much

5 participants