Skip to content

use blocklist instead of blacklist #38988

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

Merged
merged 1 commit into from
Jun 9, 2020

Conversation

shengxinjing
Copy link
Contributor

@shengxinjing shengxinjing commented Jun 9, 2020

use blocklist instead of blacklist for some reason

@shengxinjing shengxinjing reopened this Jun 9, 2020
@RyanCavanaugh RyanCavanaugh merged commit 3c49fb0 into microsoft:master Jun 9, 2020
@Helly0000
Copy link

Blacklist → crimethink
Blocklist → Double plus good!
Believe me, no one knows some reason better than me!

cangSDARM added a commit to cangSDARM/TypeScript that referenced this pull request Jun 10, 2020
* upstream/master: (36 commits)
  Update baselines (microsoft#39000)
  Make isEntityNameVisible duplicate the node builder logic to always consider type parameters as visible if they are the resolution result (microsoft#38921)
  Schedule failed lookup updates (microsoft#38560)
  Remove non null assertion on oldSourceFile.resolvedModules (microsoft#38984)
  use blocklist instead of blacklist (microsoft#38988)
  LEGO: check in for master to temporary branch.
  Deprecate reloadFs so the tests are more clear in what they are achieving and its easier to track changed behaviour (microsoft#38954)
  LEGO: check in for master to temporary branch.
  LEGO: check in for master to temporary branch.
  Make `hasCorrectArity` handle tuples properly
  fix error when use spread arguments twice
  fix(38081): allow transforming object binding to named imports
  Accept baseline
  fix names
  Fix type and simplify code
  accept new baseline
  fix rebase conflict
  add missing semi
  Add more check
  fix mission baseline
  ...
Copy link

@roberChen roberChen left a comment

Choose a reason for hiding this comment

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

spell error

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants