labofsahil/github-event-dataset-2012 huggingface dataset:
Actually I was thinking about it too, I'd say we don't need to warn people,
as nothing is really different about the matrixes. We just silently handle
the API difference and that should be fine.


On Fri, Dec 14, 2012 at 6:24 PM, Eric89GXL <notifications@github.com> wrote:

> In mne/preprocessing/ica.py:
>
> >
> > -        return to_ica, pca
> > +        if not hasattr(self._ica, 'sources_'):
> > +            warnings.warn('You are using an outdated version (<= 0.11) '
> > +                          'of sklearn.')
>
> It just occurred to me that we might want to specify why this is bad, if
> it is easy to give a simple reason. Are the decompositions better in newer
> versions or something? They looked darn near identical on my Ubuntu / 0.11
> and OSX / 0.12 versions, at least.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/264/files#r2422798>.
>
>
I get an error on Line 244, now:
```
Expected:
    Evoked (comment : Unknown, time : [-0.199795, 0.499488], n_epochs : 55, n_channels x n_times : 364 x 106)
Got:
    Evoked (comment : aud_l, time : [-0.199795, 0.499488], n_epochs : 55, n_channels x n_times : 364 x 106)
```
I think you should be able to change this in your version and it should work. I'm confused why it didn't give you an error, though... maybe you need to rebase to most recent master?
yea, so how do we deal with this difference in the OrderForm JSP?? in trunk its still searching for User in the JSP. Should that be changed to Provider in trunk?
How about you create this cache in a constructor (or a static constructor when back in ExistingOrNewVisitAssignmentHandler) and make it a final field? It will be thread-safe whereas now it's possible that this cache is created by one thread and not visible (possibly never) by another. To do it correctly your way, you would have to make the field volatile to instruct JVM to look for updates from other threads, but it's really better to do it in a constructor once and for all.
How does this deal when `params[:page]` is a garbage value? I know will_paginate doesn't play nice if you pass stupid values so I had to jump through hoops to get it a sane value.
same at line 152
wrong. The ``kernel.terminate`` is also executed if the exceptions are handled by the kernel and converted to a response
Yes, but all the components are in the same folder. Otherwise they are in separate folders.
加上translator属性吧，翻译者
¿No quisieras esperar a hacer la corrección según lo que Gabriel corrigió?
remove commented line?
missing a checkSession();
What about the situation where there is no relation (regular string field), and the field has already been set once. If I understand your code correctly, it will always show the placeholder image first, even when an image has already been set in the past.

I think we need another check that determines wether the field is empty or not?
Make sure the finally clause includes this line too or we'll leak memory if ever getattr fails.
On second thought, swap lines 45 and 46.
it is possible for localTx here to be null in the case the reply failed before creating it...
just wondering what the use is?
+1 to making the relationship between the variables clearer, but there is nothing wrong with synchronizing on `this` to maintain this relationship as they're all instance-level variables.
There are whitespaces in the ``master`` branch, but the diff doesn't seem to show them.
Rather than np.isscalar you should use mpl's duck-typing functions, like:

    if cbook.is_numlike(xerr) or (cbook.iterable(xerr) and len(xerr)==len(x)):
`->` would work here.
You added a bit of whitespace here and above. Evil.
Create a configurable number of workers
Why not just compare with an existing XML instead using t.xml_eq ? I would prefer to keep that same methodology.
So by inspection, `$attachments` is an empty array. Why the `array_merge`?
what's the split size for?
I'd like to just delete this whole formarunner deal. Those are `?subpail ?chunk`.
You mean the down method is superfluous? I don't get why sometimes the migration creates a simple back-rollable change method and sometimes it's up/down like it's '04.
Should we make this:
```javascript
var data = {
    'body': body
};
if (replyTo) {
    data.replyTo = replyTo
}
```
?
When is this variable used?
* int timer_init(
          struct timer *tmr, const unsigned int usec,
          void (*handler)(struct timer *, void *), void *data)

      Funkce inicializuje strukturu casovace pro dobu usec mikrosekund.
      Vraci EINVAL pokud je handler NULL, jinak **OK**. Casovac po inicializaci
      neni aktivni, aktivace se provadi funkci timer_start().
Coding standards: Should use FALSE not false.
sorry about that, I changed the commit and push forced again
should also be private
Are you sure ``$validator->isValid('1')`` has the correct type as "Boolean" is not a sub-type of "Boolean"?
Are you sure that the property "container" exists? Did you maybe forget to declare it?
Are you sure the method ``setUrlHelper`` can be called on ``$page`` (of type "array")?
还要加上node_modules吧？
to be consistent with the naming used in the symfony2 bundle, it should be ``load``, not ``add``.
Исправить кодировку
Tehtävässä oli tarkoitus tehdä funktio joka toimii kuten `some`, joten valmiin `some`-funktion käyttäminen ei ole sallittua tässä tehtävässä
I'll merge but we should probably change the other $stubpath just to be consistent.
No tabs in my code
oh, missed that. Thanks!
hey @slamice, the single quote at the end of this line needs to be at the front of the next line :smile:
switch it to a method and figure out if its ruby19 or 18?
So I just copy this line to requirements.txt?
<code>hg clone ssh://hg@bitbucket.org/mitar/django-missing</code>
I think both would cancel each other out. I used change in the last commit.
Move comment to docstring of the method. Of course be a bit more descriptive. It is not that it only sends an update.
I don't know. Try it. ;-) But I don't know why it would matter, if the value is  the same. It could be strange if the value would not be the same.
Same as here.
This is much much too simplified. E-mail addresses can be much more complex.
*the e-mail you provided* -> not really, there was nothing like that in the form. We could just write out again the e-mail address of the user.
Yes, maybe. But then do also registration from manually, and not as a model form.
```
{% load i18n pushserver %}<!DOCTYPE ...
```
There should be nothing before `<!DOCTYPE ...` in output, not ever newline!
Just now? Ccc.

(Yes, I noticed that you tried to take the shortcut. Thanks for spending me time pointing you to documentation.)
It's not a blocker for me.
Any reason not to just install http://pypi.python.org/pypi/PyAIML/0.8.5 ?
They are auto generated
where would the `@message` instance variable be set?
this aura has infinite duration, after cast you don't need to call it again
As the comment above says: The constants are used in code but there are no texts available in the database. So I kept the code.
You mentioned incompatibility with Gingerbread.  Should this be a compare against Honeycomb then (to include Honeycomb, ICS, and JB)?  Or maybe to be more specific, it should be <= Gingerbread return true?
No worries. If you'd like to update the pr that would be great, otherwise one of us can do it.
post fields name can be easily read from returned post_params_dict. It is no reason to ask backend provider to put them again here.
Why is so much of the indentation messed up after your change? Is your IDE doing something funky?
This should be just "2012"
This file could be deleted
I would expect 0 to be failure, not 2. Maybe write 2 for "first failed, 2nd succeeded" etc?
ok, I can "uncommit" it from the pip-requirements, and repush the branch
TODO: Bug, but benign.  Should be ".never" since we should never end up opening the file.
I'm reasonably sure that the chef_gem should be done at compile time as normal because of the intention to make it availble e.g. in the database providers, for example, [like this (postgresql::ruby)](https://github.com/opscode-cookbooks/postgresql/commit/9a6e1417a938d251522afb298903b097ad0d7ebc).
Usually I just update the .ui file and type make and it works just fine. No need to make clean before. My best guess that you're not running the version that you are compiling, somehow. You don't need to install qBittorrent to test it either:
$ ./configure
$ make clean && make -j2
$ src/qbittorrent
What should we do here? Use regular expressions instead?
yes, you re totally right.
my bad. 

I'm going to revert this one fix in my fork.
Dp you want a new pull request ?
Yes. According to the checkstyle documentation for 5.6, the module was removed from checkstyle because the idiom no longer exists in versions of Java 5 and beyond. We are using Java7, so the DoubleCheckedLocking idiom not only does not apply, but it's presence was causing errors for the eclipse checkstyle plugin since it didn't recognize it as a legitimate module.
Scary names bro.
From a user experience standpoint I would say that a spinner is more
appropriate. Progress bars should be used to display the state of a series
of tasks. In this situation we are trying to let the use know that
something is happening with the familiar spinning motion that their browser
uses and/or operating system.

I can abstract this in the followup case so that we can drop in whatever
visual notification at a later time.

On Wed, Oct 24, 2012 at 9:06 PM, Philip Schatz <notifications@github.com>wrote:

> In rhaptos2/repo/static/js/lib/spin.js:
>
> > @@ -0,0 +1,325 @@
> > +//fgnass.github.com/spin.js#v1.2.7
>
> Not quite sure how this is being used but could bootstrap's indefinite
> progress bar work instead?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Connexions/rhaptos2.repo/pull/41/files#r1936387>.
>
>
Uuuuuh.
In what way is this "for tests"? Does one have to do anything for production to stop it setting up as for a test? Or pass in anything special when using it in tests?
On v3 branch, this name based location strategy is beeing deprecated, Also for 1 class support multiple cultures will be possible.
On a like note, should this use a class identifier instead of an ID so that designers aren't limited to a single login/logout link?
I tried that right_agent changed and it didn't fix the problem per this
test ... no "WARN logger" entry in /tmp/log_test.log.

Lee

On Fri, Apr 13, 2012 at 4:46 PM, Lee Kirchhoff <lee@rightscale.com> wrote:

> If the right_agent change fixes it, that is fine with me. Here is the test
> program I was running from the right_agent repo to verify it.
>
> Lee
>
> #!/usr/bin/env ruby
>
> require 'right_support'
> require 'right_agent'
>
> module LogTest
>   include RightSupport::Log::Mixin
>
>   def self.included(base)
>     base.extend(RightSupport::Log::Mixin::ClassMethods)
>   end
>
>   def self.init_log
>     RightScale::Log.program_name = "log_test"
>     RightScale::Log.log_to_file_only(true)
>     RightScale::Log.init("log_test",
> RightScale::Platform.filesystem.temp_dir, :print => true)
>     RightScale::Log.level = :debug
>     RightSupport::Log::Mixin.default_logger = RightScale::Log
>     true
>   end
>
>   def self.log_test
>     RightScale::Log.debug("DEBUG Log")
>     RightScale::Log.info("INFO Log")
>     RightScale::Log.warning("WARNING Log")
>     RightScale::Log.warn("WARN Log")
>     RightScale::Log.error("ERROR Log")
>     RightScale::Log.fatal("FATAL Log")
>   end
>
>   def self.logger_test
>     logger.debug("DEBUG logger")
>     logger.info("INFO logger")
>     logger.warn("WARN logger")
>     logger.error("ERROR logger")
>     logger.fatal("FATAL logger")
>   end
> end
>
> puts "Initialize Log"
> LogTest.init_log
> puts "Log test"
> LogTest.log_test
> puts "Logger test"
> LogTest.logger_test
> puts "Done"
> exit
>
>
> On Fri, Apr 13, 2012 at 4:36 PM, Tony Spataro <
> reply@reply.github.com
> > wrote:
>
>> > @@ -88,7 +88,13 @@ def warn(message = nil, &block)
>> >        severity, message = filter(WARN, message)
>> >        meth = SEVERITY_TO_METHOD[severity]
>> >        raise ArgumentError, "Filter emitted unknown severity
>> #{severity.inspect}" unless meth
>> > -      @actual_logger.__send__(meth, message, &block)
>> > +      if @actual_logger.respond_to?(meth)
>> > +        @actual_logger.__send__(meth, message, &block)
>> > +      elsif @actual_logger.respond_to?(:instance) &&
>> @actual_logger.instance.respond_to?(meth)
>>
>> Rather than doing this (encoding into RightSupport the knowledge of our
>> busted SingletonPatch implementation), why not make the following change to
>> right_agent:
>>
>> https://github.com/rightscale/right_agent/pull/35
>>
>> This is consistent with RightSupport's own version of the same idea
>> (EasySingleton), with which we hope to replace the right_agent
>> SingletonPatch someday:
>>
>>
>> https://github.com/rightscale/right_support/blob/master/lib/right_support/ruby/easy_singleton.rb
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/rightscale/right_support/pull/34/files#r684593
>>
>
>
this does not work, because sinatra always sends all form fields as empty strings.
don't know exactly where, but some part if this if/else-thing will never run, because the specs only pass the expected params and not all.
just check the params hash to see that this needs some kind of `#empty?` check.
nit: add a space between : and width
You could omit the if
Sorry, been a while since I worked with that code. I meant temperature_expose. IIRC, so far we've been using temperature_expose() to handle fire reactions. What's the difference between temperature_expose() and FireBurns()?
Лишнее выводить не надо. Там ведь в файле может быть такая строка.
flush надо делать в else
Таким образом у тебя будет лишние исключение в стек-трейсе. Вынеси эти проверки за пределы try
s/searc/search
make this private and call it from within click_validate. It can be implied that you wait for the validation
Zac pointed out that is wasn't necessary, as the find_elements will do an implicit wait. I tested it and removing that WebDriverWait did not cause any problems. I think it was left over from some older code when it was needed.
Typo?
Should it be <code>self._first_address_locator</code>
I would like to add a link to the LICENSE from here but its hard to get the Org parser to realize the file type. See e.g. https://github.com/Sleft/yasnippet-latex-mode/blob/master/README.org in which the LICENSE.org is broken.
`image` is listed twice.
Comment about OME server being superseded by OMERO?
We probably don't need to check if the user is in Honors because it really doesn't do any harm if we show this to non-Honors students.  That way we only need to check in the callback function.
Oh, I see my mistake, forgot about the replacement.
Is this necessary? It is nearly equivalent to `plot_adaboost_iris.py`, isn't it?
You are right. I did not think of the default argument of ``getattr`` when I wrote this.
ok - fixed - I adopted that from ESLII
the lil conversion should be done here
I feel a bit stupid asking that but what is the difference between this method and the method below?
Also: couldn't this method use ``apply_tree`` or is there some performance issue in that?
On 10/09/2012 07:15, serge-sans-paille wrote:
>
> In pythran/pythonic++/core/list.h:
>
> > @@ -255,6 +255,26 @@
> >                  void resize(size_t n) { data->resize(n); }
> >                  iterator erase(size_t n) { return data->erase(data->begin()+n); }
> >  
> > +		T pop(long x = -1) {
> > +                	x = x%size();
> > +	                if (x<0) x+=size();
> > +        	        T res = (*this)[x];
> > +                	erase(x);
> > +	                return res;
> > +		}
>
> There clearly is a n indentation issue around there
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/serge-sans-paille/pythran/pull/12/files#r1563231>.
>
I don't have it... weird
Should probably take latitude and longitude arguments to be consistent with findAndDisplayNearby
{String} promise?
add (unit) to describe string to mark it as unit test in jasmine output
Then drop it, I said I will remove the newline. Again, is there anything else I need to change?
slice order should be 0-based
nitpick: this if could go before the parsing since we don't need to parse the args when exiting.
Un tableau de *devices* (ou types d'appareils)...
Vitalize isn't supported anymore, right? I don't think these #ifndefs are needed anymore...
I take it, this need to be removed?
`->get('Doctrine\ORM\EntityManager')` isn't supported anymore, right?
as a Numeric field
Yes, I think it would make sense.  It would also greatly simplify the spilling code - since you could just spill one bucket at a time [round robin].  
When you get close to the memory limit that would potentially allow you to not spill some buckets.
Probably. I reckon the code would be clearer and not (detectably) less efficient if you replaced the ==psc tests with calls to isPathSepChar() though
Done in  c942dca53f
It would be good to me If we keep the description as is.
So I agree to check in.  :)

Thank You!
Please fix indentation.  Pay attention to tabs vs spaces...
Typo'd `authroity`
This shouldn't go here - `Artist.all.reject {|artist| artist.displayable_artwork.empty? }` should be a class method on the Artist domain model class and here should be something like `Artist.with_displayable_art`. That logic is part of the domain model and doesn't belong in the controllers
Good suggestion, I made it so.
Would mess up WLT, f.ex.
please fix formatting (spaces before and after '=')
Destructurointi tekee elämästä vähemmän kivuliasta:

~~~clojure
(defn contains-point? [[left bottom] [right top] [x y]]
  (and (<= left x right) (<= bottom y top)))
~~~
createBuilder is about creating a child builder.

Having a dynamic parent is not allowed anymore as it was causing many issues in the Form component.
This I think would not fit any of the tracks. 
I'm going to merge the pull request, but this specific talk - don't think it works for the tracks in the conf...
So I tried this, and it crashed:
<pre>password = password.Rex::Text.to_utf8()</pre>

console output:
<pre>
msf  post(spark_im) > run

[*] Config found for user Administrator
[*] Starting to decrypt password hash
[-] Post failed: NoMethodError undefined method `Rex' for "username@im.example.com\x01HASH":String
[-] Call stack:
[-]   /opt/metasploit/msf3/modules/post/windows/gather/credentials/spark_im.rb:55:in `decrypt'
[-]   /opt/metasploit/msf3/modules/post/windows/gather/credentials/spark_im.rb:111:in `block in run'
[-]   /opt/metasploit/msf3/modules/post/windows/gather/credentials/spark_im.rb:82:in `each'
[-]   /opt/metasploit/msf3/modules/post/windows/gather/credentials/spark_im.rb:82:in `run'
[*] Post module execution completed
msf  post(spark_im) > 

</pre>
Most scanners only touch a single port, so this really is specific to the module
Are the offsets random, or can you discover them programmatically? Is there a default project? I'm just trying to figure out how a user could either make use of the description, or can figure out how to use the module with datastore options (instead of resorting to editing the module).
I need to assign it to false so the HTTP Service will start in "plain" HTTP (no ssl). Since the start_service on the Exploit::Remote::HttpServer mixin uses the datastore value directly I don't know how to avoid this without modifying the mixin:

<pre>
		# Start a new HTTP server service.
		self.service = Rex::ServiceManager.start(
			Rex::Proto::Http::Server,
			opts['ServerPort'].to_i,
			opts['ServerHost'],
			datastore['SSL'],
			{
				'Msf'        => framework,
				'MsfExploit' => self,
			},
			opts['Comm'],
			datastore['SSLCert']
		)

</pre>
I'm presuming you need this for all modules? Do you need every line to wrap at 80? Could you explain why? It would be useful for me to understand. Thanks.


On 4 Oct 2012, at 21:36, jlee-r7 <notifications@github.com> wrote:

> In modules/auxiliary/scanner/sap/sap_soap_brute_login.rb:
> 
> > @@ -0,0 +1,149 @@
> > +##
> > +# This file is part of the Metasploit Framework and may be subject to
> > +# redistribution and commercial restrictions. Please see the Metasploit
> > +# web site for more information on licensing and terms of use.
> > +#   http://metasploit.com/
> > +##
> > +
> > +##
> > +# This module is based on, inspired by, or is a port of a plugin available in the Onapsis Bizploit Opensource ERP Penetration Testing framework - http://www.onapsis.com/research-free-solutions.php.
> 
> Wrap at 80 columns, please.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
> 

email:    dave@northern-monkee.co.uk
skype:   n-monkee
ichat:     nmonkee.mac.com
web:      www.northern-monkee.co.uk
bus:       www.linkedin.com/in/nmonkee
twitter:   www.twitter.com/nmonkee
This `A` matrix has significantly more zeros than `_make_similarity`, but I think a block matrix construction would still be clearer here.
If you rebase on current master this could also be replaced with the warp function. The warp function uses a fast Cython version for order 0, 1 and mode constant, wrap and reflect.
I would break this out into a separate function. Also, here is a justifiable case for forward gotos.
That's this part I'm concerned about. Will this change break existing themes?
Actually, `toggle` is also capable of setting the state based on a flag, which is the third parameter, where we can pass the `light` boolean.  In that usage it's a bit misnamed, but this sort of "bonus functionality with extra optional parameters" business is how JavaScript programmers (who aren't us) like to roll.
> This function is lower-level than the other functions on this class.

So what does it do?  I only see the arguments documented.
> Do we cache these allocations later? Probably something we want to consider.

Yes.  Your comment is on the very line where they're cached, in fact. :)
In meters?
Should produce a warning for debug when an error occurs.
example of disconnect between "page" terminology versus "child view"
Omit "though". Also, missing space after paren.
shoot, meant to return 'normal' there.

I can return the 'unknown' constant. I was following the example set forth in getDensity() in DisplayCapsProxy.java where a medium density is returned if the density cannot be determined. Let me know how you'd like me to proceed and I'll modify the PR.
given the comment above, this assignment does not make sense
No, the script is run once by the user, and submits a ton of jobs for the regtest. Itself is not run by multiple nodes, just once, and by the user.

The line above removes the current EasyBuild installation, so a fresh one can be installed from scratch.
I'm an idiot, the incr family only operates on integers as per spec.  I removed this part from the test.  I'll try to work in the condition that disallows operations on non integer values.
TPL_INPUT and TPL_MESSAGE should be removed (any unused variable for that matter).

Since they're only used on these lines, move them into the respective spots.
Ну придумай в два слова. В `$.ajax` есть и такие ошибки: `notmodified`, `parseerror`.

Строки ок. Ты же в руби тоже символы бы возвращал, а не числа.
Confirmed. Introduced at 8aaf55c1e6216f1221569e7087bcdd1000124d5f
с тебя сочинение на тему где какой const что означает.
Spelling error: "function," not "funciton."
Better: use firstOne = true and then assign false to it.
`isNested` causes compile error for non struct/union types? Rather simply returns `false` for them is better IMO.
Do you have any strong argument about that?
Either way the point is that lines(x) is semantically a function-call.
In my defense, "enforce(!filler.empty);" was part of the original code. I'll add an error message.
Space after : (pep8)
Again, try to explain the classes purpose in the class level javadoc, not the technicalities of it. Imagine you are writing a real app.
This var name should probably change.
The difference is simple: arrays are never associative in JS
val not needed on case class.
The covariance matrix
Although the implementation is different, it would be convenient if the signature for compare is the same (we can simply ignore the salt). Also, please add information to this BCrypt class saying it is special because the salt is actually bundled with the password. :)
"le screwed"? :)
`preg_replace()` here is used with the case-insensitive flag - you don't need to list both a-z and A-Z.
I think we talked about caching the hashcode to be on the safe side?
i pushed a nice name now.
actually i think router:debug will never pick up our routes, as we do not return them on getRoutecollection.
I'm not clear that this base class deserves to exist. I don't think it's a net savings in lines of code, and there isn't a real strong behavioral or even conceptual similarity between `SuiteImporter` and `TagImporter` that it represents. Don't feel super strongly about it, though.
I get validation errors when this is a button, but none when it's a hyperlink.
This is what I put in the html validator which fails:

```<!DOCTYPE HTML>
    <html xmlns="http://www.w3.org/1999/xhtml">
    <head>
        <title></title>
    </head>
    <body><div>
        <button id="entity_user_image" class="entity_change_avatar">
           <div class="entity_profilepic_internal">
                <img id="entity_profile_picture" alt="__MSG__PROFILE_PICTURE_FOR__ " src="/"/>
            </div>
            <div class="s3d-dropdown-menu-arrow entity_profile_picture_down_arrow"></div>
        </button >
    </div></body>
    </html>```
### Array Initializers

I'd like to see some rules for method and function calls, especially regarding the number of operators - this is closely related to initializing arrays:

```
$array = array(
    'foo',
    'bar',
    'baz',
);
```

I believe it makes sense to add a comma behind the last assignment in an array, as it makes it easier to shift assignments up and down and for `diff`s only the line with an actual change is affected.

### Function/Method Calls

Concerning function and/or method calls, I've come to the following practice:

* if for a specific call the method takes only one argument, don't wrap

```
$id = $this->getRequest()->getParam('id');
```

* if for a specific call the method takes more than one argument, wrap

```
$id = $this->getRequest()->getParam(
    'id', 
    'foo'
);
```

### Wrapping Chained Method Calls

This leads me to another topic, wrapping when using chained method calls. Personal preference, maybe, but has a reason.

* when chaining, always wrap bevor the first method call and put the semicolon on a new line

```
$response
    ->setHttpResponseCode(500)
    ->setBody('Something failed')
;
```

* wrap where it makes sense, as on each level, the same object is returned (compare with above)

```
$this->getResponse()
    ->setHttpResponseCode(500)
    ->setBody('Something failed')
;
```
The w3c page on <video> seems to not care. I prefer to leave it off because it's prettier and harder to screw up.
I know, but it is everywhere in this module. As I don't know why and don't have time to investigate I just leave the current style.

By the way, **why** it is everywhere in this module?
I see what you are saying, I can remove the redundancy.
Did you try can title + summary fit in a nice way? How about the lengths? If it's title + summary, does it need more space? Is there more space to be used (ie. lengths increased)? I'll pull this into front page, but would like to consider the fuller version.
cool, that is what I though. sounds pretty reasonable.
@Freeaqingme ok, updated
このメソッドは Response を返してますが、ちょっと責任を追いすぎじゃないでしょうか。
How does this value get set when the delegate does not respond?
Can we use the same jsdoc documentation style in here?

`@param {Type} argumentName Description of argument`
The link text should ``Hooking into the JS code`` IMO.

and the file should be renamed to use an underscored lowercased name for consistency (``13-hooking_into_the_js_code.md`` for instance)
I would pull the Markdown methods out into their own utility class as I'm guessing there will be more of them, but they are (mostly) only useful to the markdown code.
Don't you need the sudo in linux as well?
I guess not. I wen updated the docs accordingly.
> "for cascading validation" would be better IMO.

I don't think "Mark a property for cascading validation" is proper english actually, that's why I changed it. Is it?
would you mind leaving the spacing intact in this commit?
make it an Interface
inroham delete kardam
I AM IGNORANT.  TEACH ME.
Make isValidRequest = False instead
Yes, good idea. But actually closurebuilder should be changed as well, to handle *.js.erb files. So we need to keep both files and create an options just to change them.
Ahhh...I think the problem lies in the conditions of the forloop. I'll hack away at it with the above model example you provided and re-open a new pull request.
Tenemos que documentar que sólo va a compilar con -std=c99
Import the using instead of using the full type name here.
"Expected" or "Unexpected"? This returns `U go now.***********` for me. Wondering whether it might also be worth demonstrating a `CanBuildFrom[StringBuilder, Char, StringBuilder]` here, although unfortunately there's no convenient `Builder[Char, StringBuilder]` around. 

Just prompted by the "Where possible, That is the same class as the current collection class Repr" in the [Seq.padTo](http://www.scala-lang.org/api/current/scala/collection/Seq.html) docs...
Please add the `type`, so proper docs can be generated
comment feels redundant
Shouldn't these be using once, not add?
No need to add this blank line.