-
Notifications
You must be signed in to change notification settings - Fork 91
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
[bug] ide extension finds no messages – unify normalPath
& normalizePath
#2184
Comments
@samuelstroschein There is no m or t syntax module defined. |
@felixhaeberle so? messages in the project exist and should be shown. the bug report is not about inline annotations |
Obviously the extension crashes somewhere because the customApi has to be set in the rendering of the messages view. Of course we could make it more modular here but in general please just setup the project correctly and then it works. The extension requirement until this date wasn't to work without a matcher (at all). For this bug, I will investigate why messages do not load and load them. It's probably nice to split the functionality here if people miss the matcher. |
The project is setup correctly? The custom API can be undefined ofc. Add an undefined check and you should be good to go. |
Again, the project is not set up to work with the ide extension (yet) because it was no requirement to have the ide extension working without the matcher, so the logic wasn't encapsulated there. This will change with this issue so that the matcher can be undefined. The added features around the message tab allow this kind of loose matcher definition. |
@jldec This is likely an SDK bug. I have successfully initialized the project but the cc @samuelstroschein I don't know why Fink is working here, but for me the SDK doesn't return messages. Is the above usage of the API correct? |
Found the reason for this. The current, not working pathPattern: Dug deeper: memoryFs makes use of the cc @janfjohannes There is already a FIXME in the code.
I will assign Jan to take over the task of unify this API. I the meantime, I fixed the example. |
normalPath
& normalizePath
will do |
@janfjohannes any update to this?:) |
@felixhaeberle i keep tickets up to date usually, no need to ping for updates unless something changes on your end eg. new info or reason something gets much higher prio etc. |
@felixhaeberle assigning you back as i dont know what the other parts of this ticket are about, feel free to reach out if i can help with something else |
Problem
Expected behavior
Displays messages
Reproduction
Other information
Fink can open the repo and project just fine https://fink.inlang.com/github.com/opral/example
The text was updated successfully, but these errors were encountered: