search

nwolverson / purescript-language-server

MIT License
184 stars 42 forks source link

Use updated state so that first jump to definition works #90

Closed andys8 closed 4 years ago

andys8 commented 4 years ago

Closes https://github.com/nwolverson/purescript-language-server/issues/89

andys8 commented 4 years ago

How to verify this change is an improvement

  1. Checkout this file / repo (https://github.com/jonasbuntinx/purescript-react-realworld/blob/master/src/Main.purs)
  2. Open editor and wait 10 seconds to make sure initialization happened
  3. Trigger jump to definition on mkRoot (line 27)

With master (0fd2cdb17cea2c76efd13b2ea76482465bde2cce) this will fail. With this PR it'll work.