labofsahil/github-event-dataset-2012 huggingface dataset:
please use another name for this
change the equal sign position
style nit: let's put the 'then' on the previous line (same thing below)
I created [BVAL-338](https://hibernate.onjira.com/browse/BVAL-338) for clarifications in the spec.
пробелы в конце строки лишние
Ugh.  Let's not handle prefs directly unless there's a gun to our heads.  Can we move all this business into `SyncConfiguration`, please?  Also, just keep a JSON-object { "history":true, "bookmarks":false } rather than nesting SharedPrefs.  Just like `enabledEngines` already does.
Maybe remove this first sentence?
It doesn't feel very encouraging to read that. Though the rest of the section sounds fine.
This wants to be above the isDatarow()
Are you sure this is a good idea? This will either

1. limit the number of patients we can create, or
2. create multiple ways of making patients.

It'd probably be good to change the previous feature files as well.
It's up to @mvriel, but I think an 80-character line width is preferable.
This should be:

`toolchain = {'name': 'dummy', 'version': 'dummy'}`
Отец и мать надо сделать вложенными объектами
for what do you need this?
Why throws IOException?
esp/esp/program/modules/handlers/resourcemodule.py:266 orders it in increasing order, esp/esp/program/modules/handlers/teacherclassregmodule.py:730 orders it in decreasing order, and it seems that there aren't any other places in the code where it orders them.

I'll merge this in as-is, since it is consistent with the ordering within the same program module.
Actually, I tried to use 3.2 sass gem when compiling.  It seems like it still doesn't work.  


On Monday, September 24, 2012 at 10:46 AM, Thomas McDonald wrote:

> In vendor/assets/stylesheets/bootstrap/_responsive-navbar.scss:
> > @@ -5,7 +5,7 @@ > > // TABLETS AND BELOW > // ----------------- > -@media (max-width: 979px) { > +@media (max-width: $navbarCollapseWidth) {  
> Sass < 3.2 doesn't support variables in media breakpoints, so this hasn't been 'upgraded' yet.
>  
> —
> Reply to this email directly or view it on GitHub (https://github.com/thomas-mcdonald/bootstrap-sass/pull/214/files#r1675608).  
>  
>  
>
I prefer the words "thicker" and "thinner"
But the second point xrstf mentioned seems valid: Using one rule instead of two different depending of the "kind of brace" feels more consistent.
Yes, but at least Java has overloading (C# I don't know). I sometimes mark "internal functions" with underscores, that would conflicts with public ones. Of course I could rename them, but on the other hand forbidding underscores  just feels little bit unnecessary too strict.
also added is a horizontal styling option.
Since its protected it could be invoked by a subclass via super(rawBuffer) where 'rawBuffer' is null thus NPE in the very first line.
Add code comment to explain this.
code smell...
We talked about this; yes it will.
I this the getClass should be return the correct class depends on the request parameter. This shoudl also check if the request is set.
Yes. unuque = !engaged
A block around the reply button might be handy
that is used in the callbacks. It's a normal javascript idom to ensure that you know which object you are working with one of the clousures below. 

Note: I didn't add the that var but I can understand why it is used in the success and error callbacks of the function.
ALL CAPS??
It should stay 0.5.5-SNAPSHOT and then during the maven release process it will get updated.
b64decode expects bytes while auth is a unicode (resp string and unicode in python 2).
It probably worked with python 2 thanks to the implicit string / unicode cast but doesn't work with python 3.

I used iso-8859-1 as it seems to be the "default" header encoding according to some RFC.
Good idea

Sent from my BlackBerry 10 smartphone on the Rogers Wireless network.

From: Jeffrey Heifetz
Sent: Wednesday, December 19, 2012 12:14 PM
To: blackberry/BB10-WebWorks-Framework; Rosa Tse
Reply To: blackberry/BB10-WebWorks-Framework
Subject: Re: [BB10-WebWorks-Framework] Fixes contact test cases for enterprise perimeter (#490)



In ext/pim.contacts/index.js:

> @@ -86,6 +86,10 @@ function onChildCardClosed(cb) {
>      application.invocation.addEventListener("childCardClosed", callback);
>  }
>
> +function isPersonal() {
> +    return window.qnx.webplatform.getApplication().getEnv("PERIMETER") === "personal";


Perhaps we should move this to utils, it seems like a generically useful function

\u2014
Reply to this email directly or view it on GitHub<https://github.com/blackberry/BB10-WebWorks-Framework/pull/490/files#r2466324>.



---------------------------------------------------------------------
This transmission (including any attachments) may contain confidential information, privileged material (including material protected by the solicitor-client or other applicable privileges), or constitute non-public information. Any use of this information by anyone other than the intended recipient is prohibited. If you have received this transmission in error, please immediately reply to the sender and delete this information from your system. Use, dissemination, distribution, or reproduction of this transmission by unintended recipients is not authorized and may be unlawful.
IMO better to nip it in the bud before it becomes a fully grown legacy shrub we have to take care of ;-)
That's the stuff we are missing on the Android side. 
Looks like it should be easy to port since it's all done through DDL/DML calls.
There are going to be problems with this pattern:

- Adding the button in the viewDidLoad event is going to be unpredictable as to whether or not it appears, because technically the view is already loaded at that point.  You're going to want to follow the pattern of the instantiation of the the other controls, where they're initialized and added to the main view in the loadView method, guaranteeing their existence (even as a placeholder) in the view.

- Because you only manage the dimensions and positioning of the button in the one place currently, it will not e.g. re-center itself when you rotate the device.  Again, I would follow the patterns of the other controls here, creating a layout method that will be called in layoutSubviews, which in turn is called whenever there's a dimension/orientation change on the view, ensuring that subviews respond to those changes.
No, This interface does not provide a callback after an async function has been completed
You no longer need the @SuppressWarnings("unused")?
please use `add_query_arg()` + spacing adjustments needed
this is wrong according to what you used previously. The second location will be `` __DIR__.'/../vendor/fos/FOS/RestBundle/src``

And I would recommend registering the ``FOS\Rest`` namespace instead of adding a second location for the FOS namespace. The autoloader is more efficient when registering the specific namespace
Once the rule is edited / removed and the size sure.
Without dashes looks more better
You can probably rely on 2.2.RC1
Restore functionality here
Which NDK do you have installed mwong?
You are using a lot of the dynamic type. C# is mostly using static types. Do you don't know the type?
Je bent zelf een Shizzle!