| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
stack test is passing all 904 tests |
Sorry, something went wrong.
Previously errorSuggestion returned `(Just "")` for UnusedImport and DuplicateImport and `Nothing` for any other ErrorMessage that has no suggestions. Both `Just ""` and `Nothing` seem to be treated the same and represented as `""` in the rendered versions. The matching imports suggestions for UnknownName now are represented as seperate suggestions, which allow for proper rendering. The JSON represenation of multiple suggestions is still a single string, as changing that would require changing the JSON schema.
to reflect the changed return type. Imports and shadowing were also cleaned up.
| Back | FazBrowse Home | New Git URL |
This PR adds the adds a Hint to the UnknownName error, if a module exports the same name. It (should) support all names including types, type classes, values etc.
Changes
ToDos
If there is interest in this, I'll implement these ToDos.
Ensuring, that the value has the correct type would be nice but seems quite involved, as none of the data structures available at the necessary places seem to have type information available.
Example:
Error 3 of 4: in module Data.DotLang.Attr.Edge at src/Data/DotLang/Attr/Edge.purs:22:1 - 23:21 (line 22, column 1 - line 23, column 21) Unknown type class Show Perhaps you want to add it to the import list? 2 possible imports were found: import Data.Show (class Show) import Prelude (class Show) See https://github.com/purescript/documentation/blob/master/errors/UnknownName.md for more information, or to contribute content related to this error.