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

Skip to content

Conversation

@robey
Copy link

@robey robey commented Mar 31, 2011

holla!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what does the non-greedy .*? do for you here? why does it need a match? Could this be simpler:

"(?:\\s+|#.*\\n)+".r

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i got paranoid when my initial version didn't work. looks like yours works fine too. changed!

@olix0r
Copy link
Contributor

olix0r commented Mar 31, 2011

Generally, LGTM.

@robey
Copy link
Author

robey commented Mar 31, 2011

pushed tweaks.

@fizx
Copy link
Contributor

fizx commented Apr 4, 2011

I guess we don't believe in java Properties?

@robey
Copy link
Author

robey commented Apr 4, 2011

We expect ruby servers will be using them too, so we want them to just be
able to use yaml.

On Mon, Apr 4, 2011 at 10:37, fizx <
[email protected]>wrote:

I guess we don't believe in java Properties?

Reply to this email directly or view it on GitHub:
#1 (comment)

@fizx
Copy link
Contributor

fizx commented Apr 4, 2011

Ah, makes sense. What's auth-specific here? Should we call this some variation of config/props/etc files?

@robey
Copy link
Author

robey commented Apr 4, 2011

In the doc (please help review! Look for an email from Bob with subject
"reminder:") we just call it yaml. It seemed like key/value pairs should be
sufficient, and that we should use something off-the-shelf.

On Mon, Apr 4, 2011 at 11:15, fizx <
[email protected]>wrote:

Ah, makes sense. What's auth-specific here? Should we call this some
variation of config/props/etc files?

Reply to this email directly or view it on GitHub:
#1 (comment)

@robey robey merged commit 33b7984 into master Apr 14, 2011
simonratner pushed a commit to simonratner/twitter-util that referenced this pull request Oct 8, 2011
smarter pushed a commit to smarter/twitter-util that referenced this pull request Aug 12, 2018
* Create a new EvaluateTaskConfig which gives us a bit more freedom
  over changign config options to EvaluateTask in the future.
* Create adapted from old EvaluateTask to new EvaluateTask
* Add hooks into signals class to register/remote  a signal listener
  directly, rather than in an "arm" block.
* Create TaskEvaluationCancelHandler to control the strategy of
  who/whom can cancel (sbt-server vs. sbt-terminal).
* Create a null-object for the "can't cancel" scenario so the
  code path is exactly the same.

This commit does not wire settings into the build yet, nor does it
fix the config extractio methods.
smarter pushed a commit to smarter/twitter-util that referenced this pull request Aug 12, 2018
Setup travis, public tests in utilLogging
smarter pushed a commit to smarter/twitter-util that referenced this pull request Aug 12, 2018
Use IO.getModified over importing the method
@nepthar nepthar deleted the authfile branch September 20, 2018 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants