Skip to content

Improve typesafety and use yarn workspaces#2

Merged
META-DREAMER merged 3 commits into
masterfrom
chore/improve-typesafety
Apr 18, 2020
Merged

Improve typesafety and use yarn workspaces#2
META-DREAMER merged 3 commits into
masterfrom
chore/improve-typesafety

Conversation

@META-DREAMER

Copy link
Copy Markdown
Contributor

Improves type safety by enabling strict mode and enables yarn workspaces to make managing monorepo easier

@META-DREAMER META-DREAMER requested a review from pacwoodson April 18, 2020 04:04
@META-DREAMER META-DREAMER merged commit 86dea11 into master Apr 18, 2020
@META-DREAMER META-DREAMER deleted the chore/improve-typesafety branch April 18, 2020 06:42

@pacwoodson pacwoodson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It seems that yarn v2 workspaces are still incompatible with React Native then Expo...

@@ -0,0 +1 @@
declare module '@walletconnect/web3-provider';

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is strange, as the package repository seems to have the file but not the npm package :/

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yea they don’t have proper types in there either, so it’s not really typesafe either way, this is just to silence the warning in strict mode

vidvidvid pushed a commit that referenced this pull request Feb 8, 2022
vidvidvid pushed a commit that referenced this pull request Feb 8, 2022
vidvidvid pushed a commit that referenced this pull request Feb 8, 2022
vidvidvid added a commit that referenced this pull request Nov 7, 2022
vidvidvid added a commit that referenced this pull request Nov 9, 2022
vidvidvid added a commit that referenced this pull request Nov 14, 2022
vidvidvid added a commit that referenced this pull request Nov 15, 2022
vidvidvid added a commit that referenced this pull request Nov 16, 2022
alalonde pushed a commit that referenced this pull request Nov 24, 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