Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Update every dependency #164

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

icecream17
Copy link

Similar to #160 but update every dependency, and to the latest versions.

This is a draft because it's not tested yet

@icecream17
Copy link
Author

icecream17 commented Apr 4, 2021

It passed? Ok, undrafting this

I suspect the tests need to be updated too, they're still on travis-ci

@icecream17 icecream17 marked this pull request as ready for review April 4, 2021 06:20
@vlad0337187
Copy link

vlad0337187 commented Sep 25, 2021

causes problem:

  • no action buttons like "import ... from ..." are present on error popups from linter / ts-server
  • were present before upgrading dependencies
Error window without import button

Screenshot 2021-09-25 at 03 59 46

Error window with import button Screenshot 2021-09-25 at 06 57 50

@icecream17
Copy link
Author

@vlad0337187
Copy link

vlad0337187 commented Sep 25, 2021

@icecream17 ,
sorry, forgot to add how it should look like:
(updated previous message too)

Screenshot 2021-09-25 at 06 57 50

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

Successfully merging this pull request may close these issues.

None yet

2 participants