-
Notifications
You must be signed in to change notification settings - Fork 30.5k
[google-apps-script] ran dprint fmt #66356
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
[google-apps-script] ran dprint fmt #66356
Conversation
|
@JoshuaKGoldberg Thank you for submitting this PR! This is a live comment which I will keep updated. 2 packages in this PRCode ReviewsBecause this is a widely-used package, a DT maintainer will need to review it before it can be merged. You can test the changes of this PR in the Playground. Status
Once every item on this list is checked, I'll ask you for permission to merge and publish the changes. InactiveThis PR has been inactive for 23 days — it is considered nearly abandoned! Diagnostic Information: What the bot saw about this PR{
"type": "info",
"now": "-",
"pr_number": 66356,
"author": "JoshuaKGoldberg",
"headCommitOid": "bf577c2507d2c817c8f768a5210335d489802e28",
"mergeBaseOid": "dfc258b892794239a47ca9bef558214109884695",
"lastPushDate": "2023-08-11T18:21:34.000Z",
"lastActivityDate": "2023-08-29T15:44:57.000Z",
"hasMergeConflict": true,
"isFirstContribution": false,
"tooManyFiles": false,
"hugeChange": true,
"popularityLevel": "Critical",
"pkgInfo": [
{
"name": "google-apps-script",
"kind": "edit",
"files": [
{
"path": "types/google-apps-script/apis/adsense_v1_4.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/analytics_v3.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/analyticsreporting_v4.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/appsactivity_v1.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/bigquery_v2.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/calendar_v3.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/classroom_v1.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/content_v2.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/dfareporting_v3_3.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/directory_v1.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/docs_v1.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/drive_v2.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/driveactivity_v2.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/gmail_v1.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/groupsmigration_v1.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/groupssettings_v1.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/licensing_v1.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/mirror_v1.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/peopleapi_v1.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/reports_v1.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/reseller_v1.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/sheets_v4.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/slides_v1.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/tagmanager_v2.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/tasks_v1.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/youtube_v3.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/youtubeanalytics_v2.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/apis/youtubepartner_v1.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script-events.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.base.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.cache.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.calendar.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.card-service.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.charts.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.conference-data.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.contacts.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.content.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.data-studio.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.document.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.drive.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.forms.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.gmail.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.groups.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.html.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.jdbc.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.language.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.lock.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.mail.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.maps.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.optimization.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.properties.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.script.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.sites.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.slides.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.spreadsheet.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.types.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.url-fetch.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.utilities.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/google-apps-script.xml-service.d.ts",
"kind": "definition"
},
{
"path": "types/google-apps-script/test/google-apps-script-tests.ts",
"kind": "test"
},
{
"path": "types/google-apps-script/test/google-apps-script-tests.with-node.ts",
"kind": "test"
}
],
"owners": [
"PopGoesTheWza",
"motemen",
"pierluigi-montagna",
"mtgto"
],
"addedOwners": [],
"deletedOwners": [],
"popularityLevel": "Popular"
},
{
"name": "lodash-es",
"kind": "edit",
"files": [
{
"path": "types/lodash-es/array.default.d.ts",
"kind": "definition"
},
{
"path": "types/lodash-es/collection.default.d.ts",
"kind": "definition"
},
{
"path": "types/lodash-es/date.default.d.ts",
"kind": "definition"
},
{
"path": "types/lodash-es/debounce.d.ts",
"kind": "definition"
},
{
"path": "types/lodash-es/function.default.d.ts",
"kind": "definition"
},
{
"path": "types/lodash-es/index.d.ts",
"kind": "definition"
},
{
"path": "types/lodash-es/lang.default.d.ts",
"kind": "definition"
},
{
"path": "types/lodash-es/lodash-es-tests.ts",
"kind": "test"
},
{
"path": "types/lodash-es/math.default.d.ts",
"kind": "definition"
},
{
"path": "types/lodash-es/number.default.d.ts",
"kind": "definition"
},
{
"path": "types/lodash-es/object.default.d.ts",
"kind": "definition"
},
{
"path": "types/lodash-es/scripts/generate-modules.ts",
"kind": "test"
},
{
"path": "types/lodash-es/string.default.d.ts",
"kind": "definition"
},
{
"path": "types/lodash-es/util.default.d.ts",
"kind": "definition"
}
],
"owners": [
"stephenlautier",
"e-cloud"
],
"addedOwners": [],
"deletedOwners": [],
"popularityLevel": "Critical"
}
],
"reviews": [],
"mainBotCommentID": 1675194228,
"ciResult": "pass"
} |
|
🔔 @PopGoesTheWza @motemen @pierluigi-montagna @mtgto @stephenlautier @e-cloud — please review this PR in the next few days. Be sure to explicitly select |
|
I'm an active maintainer of types/google-apps-script. I understand your detailed explanation.
This package was originally auto-generated by those tools but hasn't been used for updates.
Thanks. |
|
Ah great! In that case I think we can close this out and let it be formatted in the main commit that formats everything. Thanks! |
|
Actually - instead of closing these individual ones out I'm going to start merging them in individually. That way we can get early testing of formatted packages, and revert individually if we need. Could I ask you to approve the PR please? |
|
@JoshuaKGoldberg Unfortunately, this pull request currently has a merge conflict 😥. Please update your PR branch to be up-to-date with respect to master. Have a nice day! |
|
@JoshuaKGoldberg Unfortunately, this pull request currently has a merge conflict 😥. Please update your PR branch to be up-to-date with respect to master. Have a nice day! |
|
Re-ping @PopGoesTheWza, @motemen, @pierluigi-montagna, @mtgto, @stephenlautier, @e-cloud: This PR has been out for over a week, yet I haven't seen any reviews. Could someone please give it some attention? Thanks! |
|
Re-ping @PopGoesTheWza, @motemen, @pierluigi-montagna, @mtgto, @stephenlautier, @e-cloud: This PR has been out for over a week, yet I haven't seen any reviews. Could someone please give it some attention? Thanks! |
|
Inspecting the JavaScript source for this package found some properties that are not in the .d.ts files. lodash-es (unpkg)was missing the following properties:
as well as these 13 other properties...toIterator, toJSON, value, valueOf, wrapperAt, wrapperChain, wrapperCommit, wrapperLodash, wrapperNext, wrapperPlant, wrapperReverse, wrapperToIterator, wrapperValue |
|
It has been more than two weeks and this PR still has no reviews. I'll bump it to the DT maintainer queue. Thank you for your patience, @JoshuaKGoldberg. (Ping @PopGoesTheWza, @motemen, @pierluigi-montagna, @mtgto, @stephenlautier, @e-cloud.) |
|
@JoshuaKGoldberg Unfortunately, this pull request currently has a merge conflict 😥. Please update your PR branch to be up-to-date with respect to master. Have a nice day! |
|
@JoshuaKGoldberg I haven't seen any activity on this PR in more than three weeks, and it still has problems that prevent it from being merged. The PR will be closed on Sep 28th (in a week) if the issues aren't addressed. |
|
#66528 was merged, so we can close this out. ✅ |
👋 Hi! This PR is meant to let you know that we're going to be applying formatting automatically to all DefinitelyTyped packages1. This PR is split out of:
This package is particularly huge (>1MB), so we're tagging you in an individual PR as a heads up. Please post here to let us know if you have any workflow needs that auto-formatting might disrupt2.
Cheers! 💖
Footnotes
Note that formatting will be applied as a commit to
masterafter each PR. PRs won't be blocked on any CI check for proper formatting. See Using a formatter for formatting (instead of lint rules) #65993 for more information. ↩For example, if this package is fully auto-generated (rather than manually edited), subsequent PRs might have a lot of noise from formatting differences. We can exclude it from formatting in that case. ↩