-
Notifications
You must be signed in to change notification settings - Fork 29.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[gapi.client.classroom] automatic update #45165
[gapi.client.classroom] automatic update #45165
Conversation
@Maxim-Mazurok Thank you for submitting this PR! Code ReviewsThere aren't any other owners of this package, so a DT maintainer will review it. Status
Once every item on this list is checked, I'll ask you for permission to merge and publish the changes. Diagnostic Information: What the bot saw about this PR{
"type": "info",
"now": "-",
"pr_number": 45165,
"author": "Maxim-Mazurok",
"owners": [
"Maxim-Mazurok"
],
"dangerLevel": "ScopedAndConfiguration",
"headCommitAbbrOid": "42967db",
"headCommitOid": "42967dba1b32d0a27eab7dd8aeb689946a0e1845",
"mergeIsRequested": false,
"stalenessInDays": 0,
"lastCommitDate": "2020-06-12T03:19:39.000Z",
"lastCommentDate": "2020-06-12T18:05:05.000Z",
"reviewLink": "https://github.com/DefinitelyTyped/DefinitelyTyped/pull/45165/files",
"hasMergeConflict": false,
"authorIsOwner": true,
"isFirstContribution": false,
"popularityLevel": "Well-liked by everyone",
"anyPackageIsNew": false,
"packages": [
"gapi.client.classroom"
],
"files": [
{
"filePath": "types/gapi.client.classroom/index.d.ts",
"kind": "definition",
"package": "gapi.client.classroom"
},
{
"filePath": "types/gapi.client.classroom/readme.md",
"kind": "package-meta",
"package": "gapi.client.classroom"
}
],
"hasDismissedReview": false,
"ciResult": "pass",
"reviewersWithStaleReviews": [
{
"reviewedAbbrOid": "fa127a7",
"reviewer": "peterblazejewicz",
"date": "2020-06-11T09:00:28Z"
},
{
"reviewedAbbrOid": "0d1c7b8",
"reviewer": "peterblazejewicz",
"date": "2020-06-10T18:56:05Z"
},
{
"reviewedAbbrOid": "1606f90",
"reviewer": "peterblazejewicz",
"date": "2020-06-09T08:28:08Z"
},
{
"reviewedAbbrOid": "cd1fb80",
"reviewer": "peterblazejewicz",
"date": "2020-06-05T15:22:57Z"
},
{
"reviewedAbbrOid": "120658d",
"reviewer": "peterblazejewicz",
"date": "2020-06-03T08:09:12Z"
},
{
"reviewedAbbrOid": "47cecdb",
"reviewer": "peterblazejewicz",
"date": "2020-05-31T21:08:49Z"
}
],
"approvalFlags": 0,
"isChangesRequested": false
} |
bcaa6b5
to
47cecdb
Compare
👋 Hi there! I’ve run some quick measurements against master and your PR. These metrics should help the humans reviewing this PR gauge whether it might negatively affect compile times or editor responsiveness for users who install these typings. Let’s review the numbers, shall we? Comparison details 📊
It looks like nothing changed too much. I won’t post performance data again unless it gets worse. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
47cecdb
to
120658d
Compare
@peterblazejewicz Thank you for reviewing this PR! The author has pushed new commits since your last review. Could you take another look and submit a fresh review? |
120658d
to
17144ac
Compare
17144ac
to
37eeb20
Compare
37eeb20
to
cd1fb80
Compare
Updated numbers for you here from 7a61cd2. Comparison details 📊
Looks like there were a couple significant differences—take a look at worst-case duration for getting quick info at a position to make sure everything looks ok. |
cd1fb80
to
1606f90
Compare
Updated numbers for you here from f7d4643. Nice job, these numbers look better. Comparison details 📊
It looks like nothing changed too much. I won’t post performance data again unless it gets worse. |
1606f90
to
0d1c7b8
Compare
0d1c7b8
to
fa127a7
Compare
fa127a7
to
42967db
Compare
Updated numbers for you here from 58d5bf5. Comparison details 📊
Looks like there were a couple significant differences—take a look at worst-case duration for getting quick info at a position to make sure everything looks ok. |
Heh, wow - OK, looks good. |
I just published |
Can you explain your reaction, please? :) |
just a lot of bot action in this PR
…--
[·/ ] Orta Therox
w/ Microsoft on TypeScript
CocoaPods <http://cocoapods.org/> / Danger <https://danger.systems>
@orta / orta.github.com
On Sat, Jun 13, 2020 at 12:24 AM Maxim Mazurok ***@***.***> wrote:
Heh, wow - OK, looks good.
Can you explain your reaction, please? :)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#45165 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAL7DRCFFVGJHLD6W77EETRWL5QTANCNFSM4NOQZHNQ>
.
|
npm test
.)npm run lint package-name
(ortsc
if notslint.json
is present).Select one of these and delete the others:
If changing an existing definition:
tslint.json
containing{ "extends": "dtslint/dt.json" }
. If for reason the any rule need to be disabled, disable it for that line using// tslint:disable-next-line [ruleName]
and not for whole package so that the need for disabling can be reviewed.🤖 This pull request was automatically created by Maxim-Mazurok/google-api-typings-generator which generates types from Google API Discovery Service. Types were linted and tested before submission.
If there is an issue with this pull request, consider submitting a new issue.
Note you can also use these types from our
types
branch which is updated hourly.