Skip to content
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

Update request-pr-review script to latest version of octokit #39031

Merged
merged 1 commit into from Jun 12, 2020

Conversation

rbuckton
Copy link
Member

This switches us back to @octokit/rest@latest and updates the request-pr-review.ts script to the latest version of octokit.

@weswigham previously commented that GitHub revs their API frequently with breaking changes so its probably better that keep the script up to date rather than pinning the API version.

Copy link
Member

@weswigham weswigham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aye, an old version of octokit may only work for a short while.

@rbuckton
Copy link
Member Author

I mean, according to the documentation they deprecated createReviewRequest on 2020-06-05, so that was a good 5-6 days of warning...

image

@rbuckton rbuckton closed this Jun 12, 2020
@rbuckton rbuckton reopened this Jun 12, 2020
@rbuckton rbuckton merged commit b2afa7b into master Jun 12, 2020
@rbuckton rbuckton deleted the fixReviewersScript branch June 12, 2020 01:04
cangSDARM added a commit to cangSDARM/TypeScript that referenced this pull request Jun 15, 2020
* upstream/master: (22 commits)
  Small fix in `getIsContextSensitiveAssignmentOrContextType`
  Simplify `visitObjectLiteralExpression`
  Fix handling of `aruments` in the emitter
  Fix declaration emit for property references of imported object literal types (microsoft#39055)
  Fix casing for wild card keys for implicit globs to get wild card directories to watch (microsoft#39049)
  DOM update 2020-06-12
  fix(a11y): make ISSUE_TEMPLATE/Bug_report.md more accessible for folks with screen readers (microsoft#39013)
  Update request-pr-review script to latest version of octokit (microsoft#39031)
  pin version of octokit
  skip implements types with no symbols
  isDynamicName skips parentheses for element access
  Allow `e: unknown` in `catch` arguments
  Serialize (noncontextual) keyword named namespace members with export declarations in both declaration emitters (microsoft#38982)
  Patch to use this.timeout() > 0 rather than this.enableTimeout() to work with mocha 8+
  Handle missing return type nodes and nested type references missing type arguments in existing jsdoc node serialization (microsoft#39011)
  Add containerName to CallHierarchyItem (microsoft#38997)
  Fix isSameEntityName (microsoft#38999)
  Fix typo for 'blocklist' (microsoft#39001)
  remove errant tab
  Switch to isSymbolAccessible for both.
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants