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

Upgrade/reach router 9705 #10016

Merged
merged 3 commits into from
Mar 2, 2020
Merged

Upgrade/reach router 9705 #10016

merged 3 commits into from
Mar 2, 2020

Conversation

ndelangen
Copy link
Member

Issue: #9705

What I did

  • UPGRADE reach-router
  • FIX issues concerning dependencies version
  • FIX resulting ts errors

@@ -57,7 +57,7 @@
"webpack": "^4.33.0"
},
"devDependencies": {
"@storybook/client-api": "6.0.0-alpha.15",
Copy link
Member Author

Choose a reason for hiding this comment

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

This seems like a bad issue currently on next

@@ -24,6 +24,7 @@ interface ParsingResult<T> {
}

const ACORN_WALK_VISITORS = {
Copy link
Member Author

Choose a reason for hiding this comment

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

@shilman I assume the typings improved upstream from any to something specific, which might still be incomplete, OR we're doing something wrong perhaps?

I added ignores, because i assume it works as expected (test will tell?)

@ndelangen ndelangen merged commit e879b9e into next Mar 2, 2020
@ndelangen ndelangen deleted the upgrade/reach-router-9705 branch March 2, 2020 15:56
@shilman shilman mentioned this pull request Mar 5, 2020
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.

2 participants