labofsahil/github-event-dataset-2012 huggingface dataset:
But it would still be required to ensure data-consistency. If you have a combined primary key and one field is additionally OneToOne, then there should be a unique key on it by default.
Yeah, I wasn't too keen on my description either, I'll update it but couldn't think of anything better.  I'll update it.
I should mention that there's still a similar mistake in CentralBody, but all that's changing anyway soon so it's not worth bothering with.
Use `!==`.
We've to change the ManifestUri before release this one is for testing purpose
Are you able to say which ones should stay in this excludes directive?
Only when `wrap` property is set to true
Actually since we fire the event onto the tab in either condition (window is present for this tab or not) we can get rid of the else.

    if (window != null) {
      // fire event onto window first.
    }

    // fire event onto tab

Sorry about going around in circles. Didn't notice the fireEvent for tab was the same in both conditions. :\
Oh FFFFFF (I mean white, of course). It's because he put a # in the key, and we strip it out.
I like semantic decisions.  Why `uri` over `href`? :)
Let's get this done first, then review all the files and move things around a bit, or rename.
Es un error, debería handlearse. Yo retornaría bool, y en este caso que retorne false.
This is asserting nothing
Aren't `writeln` and `writefln` guaranteed to flush already?
Yes, you can execute it manually. When I was profiling the reflection overhead issue with Class#getInterfaces I first attached the profiler to the JMeter GUI, but there is a lot of other stuff going on that way as well and it is not so easy to follow as in this "standalone" testcase. During testing I infact had another lock where the JVM would effectively wait for ever in the end so that I could inspect the heap, etc ..

I left the test, because I don't think it hurts and it might come in handy another time.
normally, I would make these private.
You use new Integer(hashCode) in other places (I don't think that's actually necessary), but nit for consistency.
don't I need to also enhance the removes? Does'nt the removal of an engineering object also removes it's directly related objects (question)
Since this is the standard, shouldn't this be an else case?
The issue is that this makes 2 assumptions:

1. The WebProfilerBundle is enabled (this is not too restrictive for the dev environment as it is likely to be true)
2. This config file is loaded after the web_profiler one so that this config overwrites the other one and not the opposite. and this is likely to be false (especially if the files are loaded in an alphabetical order)
doesn't raise
That must be me, as I mentioned above :)  In addition, it's not true anymore because we hide the button when the comments tab is not visible.
Because currently, the interface does not enforce having an id field (this is on my todo-list for a next PR).
I quite like to enforce the hashcode consistency
As for type check, i thought id spare a line, but let me put that in too
Do we need similar logic in unlockMessage?
FYI, Jenkins pulls the code automatically from the Github repo and starts it's work, we don't do that manually.
Will fix capitalization. It's easy enough to change the text, let me know if you think of something better.
if we return true - could we return !true for other cases. It would provide an easier recovery path
It's possible for the user to flush a set number of documents instead of doing a global flush. That's what I had in mind. `--no-flush` is a good start, though. We can always tweak things later as users provide feedback.
if/for/while/do: Always put stuff into a curly-bracket block
Mindestlänge angeben.... sonst fühle ich mich als Benutzer verarscht ;)
Would that cause any issues with animations (if any use the mixin?) that go from 0-100 (might cause an early "flash" of 100% opacity as it hits 1) or is this mixin fairly well restricted to static bootstrap elements?
Yeah, that's a better change.

We may also one to consider the case for something like the iPad where the initial 9 notes loaded might not take up the full height of the table view so we'd want to load more automatically with any scrolling.
а вообще, лучше в цпп
the warn messgae should be moved to the org.infinispan.util.logging.Log interface.
haha.  I debated this.  Figured, if it couldn't parse the JSON, I wouldn't bother searching for the objects.  But I suppose a DB lookup is probably faster than a load from disk, huh?
Maybe the variable should be named postionDialog instead, so we don't use a double-negative. Same goes for below
How about to absorb both tags to single and just add into new `<syslog-transformer/>` component some attribute **transformTo** or **toType**
And introduce some Enum for this attribute? And the more: how about if I just want to present syslogs as String?..
Ok, fixing it as suggested by Mark. Raised the follow up issue https://jira.springsource.org/browse/INT-2497 for 2.2
Some minor changes: "the Apache web server", "the Tomcat application server", "The Cookbook".
Please try to keep the opening curly brace on the same line as the function declaration.
s/s/z/ please.
Method better named as addPatch?
nice level of detail
This compiler pass should be improved to loop over the available tags instead of reading only the first one. Otherwise, it confuses people trying to understand them and looking at bundles written by core devs as an example
i'd just remove this label, it doesn't serve any purpose given the headline already states everything
Yep, fife does that. Specifying the size in xml would also scale up smaller images however which is something i don't look forward to.
You're right. What about this? "By default is empty (``) and assumes the system default temporary directory."
This is okay for your own applications, but as a public facing API client for Bigcommerce, we’d want to keep the naming consistent with our brand, which is now `Bigcommerce` across all media, not `BigCommerce` or `bigCommerce` (`bigcommerce` in all lowercase is acceptable in some contexts).
This matches two elements on the login page, we should try to match only one so we know which one we're expecting to interact with.
I use the expanded form for this dependency because gmock includes a dependency on groovy, which is what you want when building gmock, but is almost certainly **not** what you want when *using* gmock. Suppressing transitive dependencies stops two versions of groovy from being on the classpath.
Just checking - are you intentionally doing the zipalign goal in a non-release context?  There's already an identical `<execution>` block for this in the release profile.  If you want to do it in all cases, then it could be removed from the release profile.  If you want to do it only for release builds, it could be removed here.
if you remove the commented out debugging, this is exactly the same as what we there before except more confusing.
It looks like most create* methods take ownership, but some not, e.g. createRecord.
Not part of this commit I realize.
Added a call to wizAnalytics.handleOpenURL.
On 12/17/2012 05:41 PM, Jim Porter wrote:
>
> In test/unit/xpcshell.ini:
>
> > @@ -41,6 +38,8 @@ run-if = account == 'activesync'
> >  [test_torture_imap.js]
> >  run-if = torture == 'true'
> >
> > +# partially works on activesync
> > +[test_compose.js]
>
> I'm guessing the "partially" part here is referring to the fact that 
> the fakeserver doesn't implement the SendMail command?
>

It fails in different ways depending on whether you run with a real 
account or a fake account:

- compose, reply (text/plain), forward: fails because of HTML 
upconversion with a real account, lack of SendMail on fakeserver
- reply/forward html messagelocalhost: fails because of lack of APPEND 
support with a real account, dunno for fakeserver
- reply all: uh, it looks like I forgot to change the testAccount 
declaration here, but it would fail because of lack of APPEND with a 
real account, probably works on fakeserver
- bcc self: works with a real account with the caveat we can't get bcc 
entries (not affected by HTML upconversion because it doesn't care), 
will fail on fake server because of lack of SendMail
We need to verify that this option is safe on all platforms.
`sn_highest` makes sense only for `select()`, not for `epoll()`
I changed it to "Refer to {@link #loadDrawable(String, boolean, boolean)} for more details" as this is the format for all my other links. It's the same as @see. This change will be pushed in the next commit, after you finish CR.
zIndex is now a static layout property that can be changed only by the user. Will update documentation to update this change.
Thirdly, this doesn't make any sense at all. You're already consuming the reader, which will consume the underlying `InputStream`. I wouldn't be surprised if this were hanging or throwing a `RuntimeException`.

Do one of three things:

1. Call `abort` on the `HttpGet`.
2. If you have a `Reader` over the stream, consume that.
3. If you don't, consume the entity.

Don't do more than one :)
Same thing here, please assign the object instead of assigning IDs.
Не пойму, в чем смысл.
This list doesn't display properly when viewed on GitHub. They appear on one line.
if you are using path.join to build this in line 55, you should probably use the same logic in line 59 and line 60 as line 52 and 55 respectively. And probably pull it out into a var
I think pthread_attr_destroy() should be called after pthread_create()
I have tried to mock the utils-->mixin() function with no luck. My mock is never called.
The problem is that i can't mock utils in time. It has to be mocked before it is ever required in via our code, otherwise the mock won't be called.
And I can't ensure this, because utils is used in multiple tests, not just webview.js.
Impressive use of selectors. It's very specific to the markup though, I wonder if it might be better to use a class for some of this?