Skip to content

Conversation

@wejn
Copy link

@wejn wejn commented Jul 26, 2013

Yet another way of setting options (turns out I'm not the only one needing this).

My approach allows for string/symbol shortcuts for option names.

Oh, and no, I'm not going to be adding readme and whatnot; mainly because @qrush ignores pull requests for 3 months so it'd be waste of time.

I'm just posting this in case someone looks in open pull requests and finds this helpful. I'll be using my local add-options branch, thankyouverymuch.

@raganwald
Copy link

I'm not going to be adding readme and whatnot; mainly because @qrush ignores pull requests for 3 months so it'd be waste of time.

So, if @qrush addresses this in two months, it would be a 33% improvement in response time?

@wejn
Copy link
Author

wejn commented Jul 26, 2013

So, if @qrush addresses this in two months, it would be a 33% improvement in response time?

If a tree falls in a forest and no one is around to hear it, does it make a sound?

@lrz
Copy link

lrz commented Jul 26, 2013

As far as I know, he does not owe you a damn thing.

@wejn
Copy link
Author

wejn commented Jul 26, 2013

As far as I know, he does not owe you a damn thing.

Hm, it might be because english is not my mother's tongue but I don't see this implication anywhere.

I'd be happy if you could enlighten me where I made it. Seriously.

@lrz
Copy link

lrz commented Jul 26, 2013

Sorry. What I meant is that this is an open-source project. People tend to work on them after their daily job, on a voluntary basis. When people don't reply it's usually because they don't have time, not because they ignore you. And making fun of the maintainer is not really helpful. A personal "hey, can you look at my patch" email would have been a better idea (maybe you did that, I don't know).

@wejn
Copy link
Author

wejn commented Jul 26, 2013

OK, I get your point -- I did not expect the message to be taken like this. Oops.

I certainly don't think @qrush is obliged to merge my pull req (there are two other pull reqs with similar functionality already under way).

I just wanted to make clear I'm sending this pull req in case someone finds my approach helpful and I'm not crossing my fingers for any merge; early or otherwise. And on the other hand, I don't intend to contribute more than I did, mainly because it might be in vain.

Hope this clears it up. Certainly no harm was intended.

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