-
Notifications
You must be signed in to change notification settings - Fork 57
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
Lot of errors logged on non existing resources since 0.9.0 #661
Comments
iloveeclipse
pushed a commit
to iloveeclipse/tm4e
that referenced
this issue
Jan 11, 2024
iloveeclipse
added a commit
to iloveeclipse/tm4e
that referenced
this issue
Jan 11, 2024
iloveeclipse
added a commit
to iloveeclipse/tm4e
that referenced
this issue
Jan 11, 2024
iloveeclipse
added a commit
to iloveeclipse/tm4e
that referenced
this issue
Jan 11, 2024
iloveeclipse
added a commit
to iloveeclipse/tm4e
that referenced
this issue
Jan 11, 2024
iloveeclipse
added a commit
to iloveeclipse/tm4e
that referenced
this issue
Jan 11, 2024
sebthom
added a commit
to sebthom/tm4e
that referenced
this issue
Jan 11, 2024
sebthom
added a commit
to sebthom/tm4e
that referenced
this issue
Jan 11, 2024
sebthom
added a commit
to sebthom/tm4e
that referenced
this issue
Jan 11, 2024
sebthom
added a commit
that referenced
this issue
Jan 11, 2024
Can you please check if this issue is solved with the latest snapshot version? |
I've tested with my PR #662 and it isn't much different to the version on master yet, so I assume we should be fine here. Thanks for fast fix. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I've tried to update from 0.8.1 to 0.9.0 but in our automates tests we see huge number of fails because the code now logs lot of errors during testing.
The problem is that the content type checks that run in UI thread are running asynchronously to the resource changes and if tests create/delete/move files the
HasLanguageConfigurationPropertyTester
may be queried on stale resources, flooding the log with errors.I will propose a trivial patch in a moment.
The text was updated successfully, but these errors were encountered: