Skip to content

Add .isWeakRef() - #164

Closed
iambinlin wants to merge 7 commits into
sindresorhus:mainfrom
iambinlin:feature/is-blob
Closed

iambinlin wants to merge 7 commits into
sindresorhus:mainfrom
iambinlin:feature/is-blob

Conversation

@iambinlin

@iambinlin iambinlin commented Mar 1, 2022

Copy link
Copy Markdown
Contributor

Changes as follow, Support #143 :

  1. Support ts complier for WeakRef: update typescript to v 4.6.2(latest)
  2. Support test mock with JSDOM: update jsdom to v 19.0.0(latest)
  3. Fix lint error: test code perf #163
  4. Add complierOptions for tsconfig.xo.json: fix follow error, However, the ide still reports an error. I don't quite clear which tsconfig the test file is ultimately controlled by. It seems that the addition is invalid.
    • image

@iambinlin

Copy link
Copy Markdown
Contributor Author

image

All tests can passed in my local env.I don't know why ci doesn't pass. @sindresorhus

@iambinlin

Copy link
Copy Markdown
Contributor Author

Sorry, I may have used --rebase upstream, so my commits are so ugly.

@sindresorhus

Copy link
Copy Markdown
Owner

You should always make a separate branch for each pull request.

@iambinlin

Copy link
Copy Markdown
Contributor Author

You should always make a separate branch for each pull request.

Ok, when I get home later and reopen a branch, I just arrived at the company now, ready to work.

@iambinlin iambinlin mentioned this pull request Mar 2, 2022
@iambinlin iambinlin closed this Mar 3, 2022
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.

2 participants