labofsahil/github-event-dataset-2012 huggingface dataset:
I assume that Harvest changed the naming scheme for invoices from `doc` to `invoice`. Since I've been reading the API documentation, I've never come across `doc` for anything dealing with invoices, though.

When I wrote the code to override the `self.parse` method, the Harvest API (or my brain) seemed to be having issues, and the tests were failing. I was having trouble with the differences in the response structures. The current specs for invoices run through both response structures (single and multiple invoice records) and pass with just `def self.json_root; "invoice"; end`.
Same nit as 2 above. You might just want to create a function

```javascript
function xpathTextTrimmed(contextElement, xpath) {
    var text = ZU.xpathText(contextElement, xpath);
    return text ? ZU.trimInternal(text) : undefined;
}
```

so that you can do

```javascript
newItem.extra = xpathTextTrimmed(metadata[0], 'Title_Alternative[1]');
```
We are not going to have any builds on integrity master,so no need to do cleanup here.
Probably sometime in the future Valve will allow having multiple trading
session at the same time - who knows...
On 29.11.2012 17:38, Philipp Schröer wrote:
>
> In SteamTrade/Trade.cs:
>
> > +        public bool StartTrade ()
> > +        {
> > +            // try to poll for the first time
> > +            try
> > +            {
> > +                Poll ();
> > +            } 
> > +            catch (Exception e)
> > +            {
> > +                if (OnError != null)
> > +                    OnError ("There was a problem connecting to Steam Trading.");
> > +                else 
> > +                    throw new TradeException ("Unhandled exception when Polling the trade.", e);
> > +            }
> > +
> > +            StartTradeThread();
>
> Why would you ever want to share a |TradeManager|? A bot will only be
> able to have a single trade, this would make things only more
> complicated.
>
> In my opinion, these should be seperated because:
>
>   * A |Trade| is *only* responsible for adding a nice API to Steam
>     Trading, which means it's essentially a wrapper around the API.
>   * A |TradeManager| creates new Trades and manages the threads for
>     them. Although stopping a |Trade| would be a /bit/ more
>     complicated, I think it supports a clean API. Canceling would just
>     happen with a |Cancel()| method which would emit a |TradeCancel|
>     event. The manager listens for that and terminates the thread if
>     necessary.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Jessecar96/SteamBot/pull/102/files#r2267042>.
>

-- 
Sincerely
Richard Schwab
Should this assert something?
This class likely won't ever have unmanaged members, why is this here?
Good point! +1
Define it in the Y.eZ namespace!
On Sun, 22 Apr 2012 17:11:38 -0700
Kouhei Maeda <reply@reply.github.com> wrote:
> What do you think is better I think next? 
> > class JSONConverter:

 Please use new-class style, don't forget object.
 You can write class without object but it's just for a backward 
 compatibility.


-- 
Regards,

 Hideki Yamane     henrich @ debian.or.jp/iijmio-mail.jp
I think the waits should always immediately follow the relevant action that's performed. In the case for the mentioned test this would be when the home page is loaded (so the home page is not considered loaded until the navigation menus are all visible). There may even be something that loads later that would be worth waiting on. For now, this is probably a reasonable fix, although I suspect this will be checking that the elements are all present rather than visible, so there may still be a race condition.
I think the same could be done with a plain Ruby class though.  When the class is first created it could read in all the config files.  Anything else that is set afterwards like the CONF_NAME and CONF_DIR for each plugin could just be registered with the class.  Just a thought.
No i just tested this, and `rm_r` is not a valid method for an instance of Pathname.
This will get the comments for a content item, this won't be used?
That's not exactly what I mean. Managing roles for groups should be done entirely through the group functions of the API, which is what this is enforcing. There would be no problem assigning a role for somebody onto another user -- this may be how private access to user profiles are granted?
I'll go with `getTenantConfig` since we might want to save `getFullConfig` to retrieve tenant + other configs.
Is there a way we can do this without using threads?  If not, that's OK, but, the server uses Tornado which is event driven.  I just checked out pybonjour and it looks like it can work with select (which is how the ioloop in tornado works internally)

Please check out http://www.tornadoweb.org/documentation/ioloop.html

Here's what I think you'll need to do
```python
io_loop.add_handler(service,  some_callback_to_dnsserviceprocessresult, io_loop.READ)
```

This means you won't have to run a select loop yourself.
Камменты такого типа бессмысленные. Они требуют времени чтобы их прочесть, но не несут в себе полезной информации
Даже если не осмысленные названия переменным, так хоть комментарии в коде делайте.
--
Your sincerely,
Sergey Lebedev.

Twitter: @drasgo
Facebook: s.a.lebedev
Это я согласен, но есть чем возразить )во-первых, тут были две другие реализации метода, полностью рандомные, но вы сказали, что это интереснее ) во-вторых мутация никогда не бывает рандомная, все подчиняется своим законам ;). Согласен в общем.
Категория должна быть вне класса
Вместо [self.DNA insertObject: atIndex:] в данном случае эффективнее использовать [_DNA addObject:]
I've checked both methods by uninstalling ghostscripts (so the conversion doesn't work). None of the methods seems to work (the flag isn't set, and no exception is raised.). We could check manually that the pdf files aren't converted but that seems like a terrible hack to me.
Good to know :) thanks Alta189
Ok. I will change this to ```@ShardKey```
Ah, nevermind. I misunderstood how the parameters were being used. This method is rightfully abstract in the base class :)
Can't we just say:

> If called from a block running on a RACScheduler,
We don't need this line anymore.
What is this supposed to test that is related to song type? I don't see anything about types here?
Hmmm. Something may be fishy here. Checking a tag out *should* detach the HEAD.
I can live with "subject='fsaverage" but would rather avoid "subject=None". It's obvious that it's fsaverage looking at function signature and does not require to read the docstring.
Add some javadocs here and @since annotation saying version 10
Does this result also need to be converted to utc for safety?
Thanks! Do you need me to correct them?

Carolyn

Sent using a mouse-sized keyboard with feigned autocorrect intelligence.

On Dec 25, 2012, at 10:22 AM, Mike Fiedler <notifications@github.com> wrote:

> In soft_skills.rst:
> 
> > +Translating Customer Requirements
> > +---------------------------------
> > +Operations roles are customer service positions. These careers
> > +require a level of customer interaction because the services delivered
> > +by the Operations professional must be driven by customer needs.
> > +In this case, customer is used to mean the business, organization,
> > +or other entity that is employing the Operations professional. Some
> > +questions to ask to help the Operations person understand requirements
> > +from the customer perspective:
> > +
> > +* What is the core mission of this organization?
> > +* How does Operations support the mission?
> > +* What does the organization need from the Operations professionals?
> > +* Why should this organization come to these Operations people for this service
> > +* or solution? (What is the value proposition for Operations within this organization?)
> > +* How could Operations provide more value: higher level of competitiveness, faster service delivery, stronger security, or other benefit that aligns with the mission?
> You can break bullets into multiple lines in raw and preserve the bullet by doing something like:
> 
> * This is the first line
>   This is a continuation of the first line, indented 3 spaces to be inline with the previous line
> —
> Reply to this email directly or view it on GitHub.
>
Updated, hopefully with a better example
I agree that it is quite difficult to search in the current standards. You're right, but there's also plenty of places in Doctrine and Assetic that do as I did here. 

No problem, I will use the most common Symfony2 way with no spaces :)
yeah, the blog post was written at the time of sf1 :)
文件以配置中指定的顺序被nginx检查。
Rad, thanks.
I went with the name NpyIter_ to be in the spirit of the numpy-refactor branch. Over the long term, it would be nice to remove all Py* names, because everything Py* should belong to CPython itself, and NumPy can claim everything Npy*.
You should define this like I said, as `SymPyDeprecationWarning("message")`, so that you can reproduce it from the string.
s.append('*')?
Это лучше сделать не ссылкой, а просто константой.
Плюс, константы именуются с первой заглавной буквы.
Плюс, хардкодед-цвета — говно.
core.h после основных инклюдов же.
`.content.pics-group-spots`
it would be more flexible to inject the environment and use the DI to configure it, as it would allow modifying the Twig instantiation (to add extensions for instance)
Can you check your line ending settings? A lot of fake changes going in.
duplicate use
Perfect, no problem, solved :)
ok, i'm just saying there's a simpler way to write ```(if (not (nil? x))...)```

it's ```(if x ...)```
Prefer interpolation to String#+
same for implode http://twig.sensiolabs.org/doc/filters/join.html
Shouldn't these be $java::config::jdk_oracle_java6 ?
I actually think there is a problem with this method. Not far after this code, there an::

     elif callable(match):
        matchfunc = func

``func`` is not defined anywhere in this method. I'm pretty sure it is meant to be ``matchfunc = match``.
This is the standard rms, right? So another step would be needed to scale the rms to have a standard magnitude irrespective of the number of pixels?
Looks good, but can we format this slightly differently for readability reasons:

    waitFor(
      function () { marionetteScriptFinished(true); },
      function () { return manager.connection.status === 'connected'; }
    );
I think you can resolve the issue of pre-existing alarms by replacing line 41 with:

self.wait_for_condition(lambda m: len(m.find_elements(*clock_object._all_alarms)) != initial_alarms_count)

This kind of renders the following assert obsolete, but it's still good for understandability, imo.
I previously had this in here because of the camera lock test but as I was blocked by another bug in the app I removed the camera locktest and submitted the pull. I'll remove it now :)
“vermindert” moet ook met een “t” en niet met een “d”
Myeah.. I was pushing the old hash before the rebase. Should be ok now
And it's connecting to the db correctly? Goliath finds it's config file by looking for a file with the same name as the api file in the config directory.  Alternatively, you can give it a -c (i think) flag to pass in a config file.

Try putting a puts statement in the config/database.rb file to see if it's getting loaded. If it is, then I missed a commit somewhere, heh.
Should this just be list (instead of data list), since GridField just needs a SS_List?
the rest looks fine but why do we need this? what about node's regular assert.deepEqual() stuff
Elke poging om 'm Nederlands te maken wordt krom. Permalink is de meest neutrale en we overwegen dit voor WordPress zelf ook aan te passen voor versie 3.6
umask is a bad idea to use as in threading environments it affects other threads too. Especially you are never restoring it back to its previous value.

http://us3.php.net/umask see the note.
it is not ``just before`the route callback as the ``before`` event occurs between them (and the ``kernel.controller`` event btw)
Perhaps better as "Structure of test directory". Don't want to get too technical or detailed in a title.
I think "Mac OS" (with a space in between) is more appropriate. Same for line 66.
SUCX
Is this comment still valid? I don't see anything about forking ?
I found this sentence to be awkward. "looked back the way the mules strained for..."
Might just be me.
this is probably better written as just

```javascript
if (item_data.deadline){
```
You need a period at the end of your first sentence :P I'll do a scan for what I think is reasonable, but in critiquing grammar it would be helpful to make clear, unambiguous statements.  It took about ten goes to work out the correct phrasing of "I think you mean an not and and also no period please" to make sense of it:

`I think you mean "an", not "and", and also no period please`

would have been better ;)
And spaces again.
I've updated PR and I've decided to extract it to private method. Is it OK?
Whatever you think is right here, go for it. My personal preference would be to just write `(config.services || [])` but I'm not religious about it. No point wasting a discussion thread on it.
You have to fix this. ;-)
Thanks Jack.
Maybe the `nm` command is good candidate for "testing" libraries, assuming that if `nm` can enumerate the contents of a library, it can be linked.
Better would indeed be a test program, preferably one that actually tests the functionality in the library, but there is not a universally adopted convention for doing this yet.

> 
> Usually the executable, if the formula provides one; many just do `foo --version` to check that the executable actually works, though some actually have nice tests that actually exercise program functionality.
> 
> Libraries are harder to test. It would be nice to compile a test program to ensure the lib can be linked, though I don't think any formula do this currently. We usually just skip the test on things that only build libraries.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/mxcl/homebrew/pull/10398/files#r477780
You mean this: https://github.com/reddit/reddit/pull/341/files#diff-2
Nope, I missed it in the superclass. removing
if (basePoints0 > 0) -> if (basePoints0)
Would it be possible to simply say "9 years old today"?
btw, if you want to add it for all entity managers, it could be simpler to omit the connection name and let the compiler pass do the job :)
Do you not need to reassign to_shift to the result of this expression?
you're right, I prefer this. Thanks.
my eclipse does not like "rawtypes" apparently javac does not support it either.
No. _name is the name to use in the database, for keys. It's computed by joining namespace and __name__, and must be unique in the database.

In this case, we found that the computed _name is not unique, so we inform the author about that, giving him informations on how to find the problem. We don't give him "namespace:classname" because he don't have to know how the internal _name is computed.
good, point will consider that when improving the test suite configuration improvements
Change to max_depth. Dpeth => Depth.
@p01, yea, that might be an idea, thanks!
Here's the boolean `element.hasAttribute('data-is-response')` ;)
You mean libapr, right? I think it only uses sysv shm when there are no better alternatives available, like posix shm or shared mmap.
Why would you remove this cache variable? Now you're calling diskCacheSize() twice.
See the class doc-string, this is just a sketch that I'm keeping around until I can try to unify this with say, something like Tobit and the nonparametric stuff.
Still not sold on the name "Ago" - what do you think?  We could do something like "TimespanFormatter".  I'll leave it up to you.
Looks like this revision is no longer in the `master` branch of your fork. Its `master` points to `5f1733b82c065ba516e4d97eefcdde8a8427f941` now.
does this mean we can't override them anymore? that it will always be localhost and 8080
This is unrelated & probably should be merged independently.
What is this? HTML4.01. Gross.
It is good enough for now.  Later we can tweak it to remove / add the button as needed or something like that.
Actually, we bundle the RNG stuff, so we don't need it for that.  But it might not hurt to suggest doing --with-math anyway.  Either way, if someone hasn't done so already, it might be worth checking if such a restricted installation is sufficient.  Do we know that to be true?
Would it be useful to add at the end the code that is in some of the other test files, i.e.
```
if __name__ == "__main__":
    test_Cubic_spline()
```
(etc.) so that the tests can be run individually while trouble-shooting issues that might arise in future?
Would be nice if this error handling wasn't so much copy/paste.  Refactor these two mail functions at some point?
Do we really want to close the caller's output stream for flags.out and flags.err? That's not normal for Java APIs, is it?
What was wrong with this line?
See comment above about more validation. We might get to this line with the cert file path not having been set and not having been copied.
I think it was Ruby that made me cave in the end. A few too many times -ise rather than -ize causing tests to fail and the like.
Will do - but it wasn't documented. Added an issue for you for that now.
Ah, ok, don't use `$('.browserid-login').on('click', [...])` then; it's not supported that far back.
Yeah, actually that would be good. Fixed. I usually try to avoid truthiness tests, but in this case it's probably the right thing to do.
Why are restarts being introduced in lots of places? This almost certainly indicates breaking changes for clients.
This is very likely to break. You should detect the lines you want to remove instead
May I suggest flipping the order to `OSCAR_ALERTS_EAGER` so prefixes provide natural domain grouping?
This is indeed much much better :)
what about |var contact = this._contact| and use it?
is there a space/tabs problem here?
Why are you changing this part?
By default installations are in English, so this would not make a lot of difference.
Back to protected.
I can agree on either way of doing it but i think whatever it is it should be consistent within the source tree.
BoardManager is a singleton. As for additional plugins for new boards,
it's up to the creator of the board to work on this. And massive advantage:
we can tell board makers they are only allowed to work on their plugin, and
not touch a single line of the GCS ! Basically: all gcs-related board
support is now located in one single plugin per board manufacturer, no
touching the GCS code. Think of it: right now, there is PipX specific stuff
in the UAVTalk plugin! That's real bad...


On Tue, Dec 11, 2012 at 12:16 PM, peabody124 <notifications@github.com>wrote:

> In ground/openpilotgcs/src/plugins/boards_openpilot/coptercontrol.cpp:
>
> > @@ -0,0 +1,71 @@
>
> I think the problem right now is if you want to support a new board you
> have to make changes in 20 different little places. Hardcoded VID/PID (and
> only one at a time) being the least of the problems. Adding a hardware
> page, making the bootloader know about it, etc. If we start supporting 10
> different boards we should try and make the code specific to each be
> grouped together instead of all throughout the code.
>
> Would you be happier if we make the plugin singleton :D
>
> As to things being done dynamically that could also be done and would be
> worth it. In this case based on some external files (xml ?) the plugin
> could create and register different IBoards so that is certainly not
> exclusive with this IMO.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/PhoenixPilot/PhoenixPilot/pull/79/files#r2384014>.
>
>
James...
Trailing comment is a bad practice.
I would expect that an implementation of Map be used that maintains the order (since we are passing steps in via a List).
Used on (testSaveWithNoPropertyToSetId)[https://github.com/danielpassos/aerogear-android/blob/configurable-id/test/org/aerogear/android/impl/datamanager/MemoryStorageTest.java#L95]
What do you mean? Is it because I'm adding comments about the links themselves?
Same reasoning here as for the changes I made in defer-resolve.js
This isn't a git submodule already? Shouldn't you be able to do a recursive checkout on the main repo to get these?
I don't know if this changes anything, but wouldn't it better to output `0.` if `x < -30` and `1.` if `x > 30`? Also, since this very particular function is at the core of the algorithm, a Cython version may be a better choice.
`if precompute or ...` + wrap everything in parenthesis.
les deux, et un typedef bien senti fera le travail dans un header
не нужно все проверки в одном тесте делать. здесь достаточно было поправить тест. а проверку доступа сделать отдельным тестом
DMD "coding convention" - `isOverloadable` already exist in `Dsymbol`..
Yeah, I thought I provided tests..
I will try to fix that..

Le 10 avr. 2012 à 20:34, Jeremy Mikola<reply@reply.github.com> a écrit :

>> @@ -16,6 +16,11 @@
>>         "symfony/console": "2.1.*",
>>         "ruflin/elastica": ">=0.19.0"
>>     },
>> +    "suggest": {
>> +        "doctrine/orm": "2.2.*",
>> +        "doctrine/mongodb-odm": "dev-master",
>> +        "propel/propel": "dev-master"
> 
> Oddly, the test suite passed with `propel/propel`. I suppose that means we really have no Propel test coverage.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/Exercise/FOQElasticaBundle/pull/92/files#r667047
This does not answer the question.  is you're answer relating to Android or Iphone or BB?  Please specify.  Have you tried it on another platform?
Après téléchargement des sources , vérifier les ...
Just thinking.. maybe we should make this `Map<String, List<Element>> originatingElementsByGeneratedClassQualifiedName` a separate class, that handles the creation of list when there's no entry, etc. It would have a shorter name too :)
Might be clearer with a blank line before this comment.
This constant really should be in a header file - otherwise you can't check to see if the value is matched.  Can you move to defvalue.hpp please.

If we add more exceptions to this dll then the range of errors should really be include in system/include/errorlist.h.  Don't change for this one though.

Also we have tended to use #defines for error numbers
e.g., see hqlerrors.hpp

Otherwise I think this needs to be a const static int to allow the compiler to optimize it.
It looks as you're only using args if ```same``` is false, but you always link them. Shouldn't this code be down at the bottom to avoid this?
```KNOWN_DATATYPES``` perhaps, or ```SUPPORTED_DATATYPES```?
duly noted and amended
Shouldn't this be:

```
urllib.quote(v.replace('%0A', '')) 
```
? 

Otherwise it's quoted before the removal happens:
```
>>> v = '%0A'
>>> urllib.quote('%0A').replace('%0A', '')
'%250A'
```
Ah! That explains the conflict when I rebased. My bad. Will remove it.
What is broken about the `glpk` formula? I get a `libglpk.dylib` and `libglpk.0.dylib` installed in my `HOMEBREW_PREFIX/lib`.
Either way - I'll squash everything into one commit before I merge.
OK, so delete the two "intransitive()" and everything else is the same?
Nit: Add a blank line to be consistent.
ok now i see what you meen, sorry
in your last post i saw line 150-151.

this line is for users with php zend module loaded.
the name attribute "module_Zend Optimizer" of an anker-tag is not xhtml valide, so replace it with "module_Zend_Optimizer"
could simply test !=
I'm not sure about the name of the class. Class names are generally nouns, not verbs.
It's for destructing `env` before calling llvm_shutdown().

Rubinius SEGV-ed without this indentation when exiting with unusual conditions (interrupt by Ctrl-C from the console). So I guessed `Environment` destructor is touching llvm objects/functions after `llvm_shutdown()` is called.

For example, without this indentation, I couldn't clean-build rubinius.

Should I explicitly mention this indentation purpose? Or, Should I put the whole indented code into another function... Is it better?

Thanks for reviewing!
On joo. Mieltymyksistä ja tottumuksistahan tässä on kyse. Saatat kyllä huomata seuraavan kerran kun tungen koodia repoon, että nuo sisennystason muuttuessa olevat rivinvaihdot ovat kadonneet mystisesti. Mielestäni sisennystason muutos ryhmittää koodia ihan tarpeeksi, niin että ylimääräinen rivinvaihto siinä kohtaa vain hidastaa koodin selailemista.
Tietenkin voisin yrittää kuvitella sinne aaltosulkeita? :P
Ok for /tmp
For the ipaddress, I have made all needed "require/notify/refreshonly" to regenerate "razor.ipxe" each time razor_server.conf have been changed
And razor always use the ipaddress in the config file to generate the url for razor.ipxe.
So the ipaddress in razor.ixpe is always correct.
wait....... what are you trying to rescue here?
disconnect_udp will do it for you
use the methods from ```Post::File``` which are platform and session type independent.
uh, pass_hash = nil?
If you removed this special case, would that mean that `an_image_collection[1:2]` returns a one-element `ImageCollection`? I think that would be more consistent, since `a_list[1:2]` and `an_array[1:2]` return a single-element list and array, respectively.
We shouldn't need this file. Can you try without it?
Same comment as _getCollection
And an @example for using it with the camera.
Perhaps make a function for sphere intersect IDL or at least add a comment about what this is doing.
I am proponent of leaving select the way it is for now.  Here is a one or a couple reasons why I like this implementation: http://strd6.com/2010/11/select-and-reject-useful-javascript-game-extensions-22-and-23/
R# uses inside. VS uses outside… it's debatable at a minimum :)

But yeah… maybe we should follow VS as it's more MSFTish.

From: Chris Tavares <notifications@github.com<mailto:notifications@github.com>>
Reply-To: WindowsAzure/azure-sdk-tools <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Wednesday, October 24, 2012 1:05 AM
To: WindowsAzure/azure-sdk-tools <azure-sdk-tools@noreply.github.com<mailto:azure-sdk-tools@noreply.github.com>>
Cc: Andre <andrerod@microsoft.com<mailto:andrerod@microsoft.com>>
Subject: Re: [azure-sdk-tools] Adding Restart-AzureWebsite command (#497)


In WindowsAzurePowershell/src/Management.WebSites/Cmdlets/RestartAzureWebsite.cs:

> @@ -0,0 +1,83 @@
> +using System;
> +using System.Collections.Generic;


Ok, will change. But I think we should revisit the convention - we're fighting our tools. Either that or we need to get the VS "Add Class" item template to put the usings inside the namespace.

—
Reply to this email directly or view it on GitHub<https://github.com/WindowsAzure/azure-sdk-tools/pull/497/files#r1923347>.
It may be, but I'm not sure how to approach it from that angle. The search should be quick since its on alive windows, which shouldn't be more than 10 in most cases. Also keep in mind that this is a temporary solution to fix the TabGroup focus problem.
How can you tell if the Blob has binary data vs. base64 encoded data (which in practice would be base64 encoded twice)?
This destroys all cookies, not just cookies from the specified domain. I opened a new feature request for this issue: http://jira.appcelerator.org/browse/TIMOB-11570. Closing this PR.
Basic Auth does not Url Encode, and I'm happy to not care about this, so the case wont match and you'll get UserCredentials.missing if you use a password with a ":" character.  That'll teach you to have a stupidly secure password :)
There's a bunch of these forEach callbacks, essentially all doing the same thing, except for the data property. Could refactor that, something like this:

```js
].forEach( add( full) );
```

Or even:

```js
add( full, [...] );
```
не знаю. я так и не понял чем мое "выравнивание по смыслу и схожести колонок" хуже чем "выравнивае по тупому следованию правилу"
The only 'bad' thing here is the little indirection it takes to get to `TypeInfo.init().ptr`. The `typeid(T)` itself compiles down to an address, so it really shouldn't be that bad.
`_ensureStatDone` was quite a bit more descriptive. Why change it?
It's not a big deal, but for debugging purposes, it would be better if you provided `s` as a message for each of these assertions. For example,

    assert(findSkip(s, "語") && s.empty, s);

That way, when they fail, you see what the actual result was without having to edit the code. Since, I assume that they're all passing right now, it's not a big deal, but it would make debugging easier later if a change breaks the unit tests.
Is the XML tag not taken care of in the layout/view?
The method type parameter  `T` is hiding the class type parameter. I think it can be removed.
is this correct? it seems to be creating a render buffer of type stencil and attaching it as a depth attachment?
Yeah, this all will go away once I get a chance to convert the collection data processing to something more like how synclets work, they're registered as exported callbacks, for now it's just a transient old format
There isn't a function `Data.Text.breakBy`, however, I think I know what you mean. One can simulate it like this:

    breakBy = X.grupBy . const . fmap not

Also, `intersperse` isn't what you're looking for, `intercalate` is. (`intersperse` is `Char -> Text -> Text`, `intercalate` is `Text -> [Text] -> Text)

You are right that it would be easier to read (I think.) I'll benchmark it real quick.
Agreed. But there's also "using namespace llvm;` to detect when using the other method.

The good thing is that once done - we could recompile any LLVM application without changes (and maybe even submit this patch to the Houdini folks ;)).
Could you also add an example about how to search for NOT NULL:  It took me
far to long to discover the answer is something like

public function published()
    {
        return $this->filterByPublishedAt(null, Criteria::ISNOTNULL);
    }

Is there a link to a page showing all the Criteria constants?


On Tue, Oct 30, 2012 at 11:08 AM, William Durand
<notifications@github.com>wrote:

> In reference/model-criteria.markdown:
>
> > @@ -74,6 +74,20 @@ $query = 'SELECT book.* from `book`
> >  WHERE book.TITLE = :p1'; // :p1 => 'War And Peace'
> >  {% endhighlight %}
> >
> > +* You can control the filter a Criteria parameter:
>
> Better: "You can use the second argument to define the condition:"
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/propelorm/propelorm.github.com/pull/173/files#r1979528>.
>
>
@lsmith77 is this the desired behaviour?
И ты не знаешь, откуда приехал таб?
What's that for? (@mattvague)
We should probably keep the `(` on the same line as the conditionals to improve readability.
We should be using the regex from our jquery.validate plugin here - it is much more robust
Yep, that's what I wanted. I should read more.
Or actually in this case, there really is no need to call `#to_s` anyway since `update_attributes` accepts symbols.
I think "Validate a manifest response." is fine.
yes. Btw, PHP-CS-fixer should be able to fix it for you here.
cout << printf
;-)
I just added back the named param (see above).
I do not understand also: is not also "normal" list indexed?
Tuple?
i wasnt sure what the dialog button was for, if exercises or dialog...
Okay I will put the row back.
Won't this break if one of ipaddr or IPy isn't installed and netfields is in installed apps? How should we handle this type of case?
В BaseContext
gr8 change
We prefer not to use `it` blocks like this. Instead, we generally follow this convention:

```ruby
subject do
  define_model(:example, :attr => :string, :other => :integer) do
    validates_confirmation_of :attr
  end
end

it { should validate_confirmation_of :attr }
it { should_not validate_confirmation_of(:attr).with_message('custom message')
```

etc.
and propel ?
только этого не надо...
* No it won't!
Remove, change to JSON
var is not a dynamic type (http://msdn.microsoft.com/de-de/library/bb383973.aspx vs dynamic http://msdn.microsoft.com/en-us/library/dd264741.aspx) 
so, var is an implicitly defined type - I'm using it for temporary variables because it saves a lot of writing and looking up for the exact definition of the other type, and has the same effect (the type is checked and defined by the compiler and there should be no difference in the library if you use var or the exact type)
Hardcoding?
I substred name (in yaml or annotation or xml) of association mapping.  
There is a more reliable method of forming a alias?