labofsahil/github-event-dataset-2012 huggingface dataset:
Really guy! It make more sense!

An insert alias ok ?

On Wed, Aug 15, 2012 at 1:05 PM, parkeryoung <notifications@github.com>wrote:

> In lib/google_calendar_oauth2/calendar.rb:
>
> > @@ -17,8 +16,29 @@ def self.find(query)
> >        @cal
> >      end
> >
> > -    def self.create(attrs)
> > -      connection.execute(api_method: client.calendars.insert, body: [JSON.dump(attrs)], headers: {'Content-Type' => 'application/json'})
> > +    def self.insert(attrs)
>
> Oh, to match their convention, I can see that. I think it is okay to have
> insert as an alias, however I would prefer for create to be the name of
> the actual method and insert to be the alias.
>
> My goal was to sort of mimic rails convention (Though you don't have to
> use rails), and I have seen create used much more frequently than insert.
>
> Also it seems that english isn't your first language, but Calendar.createmakes a lot more sense than
> Calendar.insert in english.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/parkeryoung/google_calendar_oauth2/pull/4/files#r1383770>.
>
>



-- 
*Peterson Ferreira*
Celular: +55(41)9145-0982 <petersonferreiras@gmail.com>
Skype: peterson.fs <skype:+peterson.fs>
Site: about.me/petersonfs
Now I have tested all collectionable, the loading mock models doesn't work due to fact that usage of the ClassRegistry::init() had been changed.
Therefore, fixing all tests is required before applying this change.
Please wait a moment until all tasks are done.
s/multible/multiple
Mind upper-casing this while you're at it? :-)
formatting is off for this method
The name confused me a bit, maybe we should call it 'kExpectedSymbolsProcessedPerMs'
There's mockito-core in the parent pom. Replace mockito-core with all of these test libs in the parent pom.
For now, we can leave the code as it, but it will be better to inject the client inside this service. So we can add logger to the client without changing the current code.
I get your point too and this can work and also add more options for advanced users/usages.
Maybe I misunderstood the term.  What would you suggest?  I didn't like isNonCalendarTimeUnit.
This should be written in English:

> Load online users
Please, do not just write, that tab should not be opened by right click. Simply load all data needed and display full page, or redirect user to specified page, where wanted tab will be displayed (what is probably not covered in the code right now).
seems like you can easily do this now?.
Please do this for the clientWebview as well and remove this line.
It should work on Mac too?
This should be orderedDict, not just a dictionary. The order here is important. Check in collections.
Should be in plugin - not global namespace
Don't you still need the label decorator?
> It's also a bit wonky working with the Channel API with external JS.
How so?