- This topic has 62 replies, 7 voices, and was last updated 7 years, 4 months ago by support-piotr.
-
AuthorPosts
-
support-piotrParticipantGorand,
If you have moved files around, you need to update “require” paths as well:
require(“../index”) => require(“tslint”)
require(“../language/utils”) => require(“tslint/language/utils”)since you cannot refer to any module within tslint package relatively if the file is outside of the package. That’s the first step.
Second problem is in the way node.js resolves module locations. Any required modules from your custom rule script would be looked up based on the script location. That means if none of parent folders of folder containing your custom rule have
node_modules
with required module (be it tslint, or path_sync, or whatever else), the code will fail to load even if tslint was initially loaded fine. I will soon come up with a solution for this and give you steps to patch Webclipse without need for a new release. For now, a workaround would be to execute following commands in the folder, which contains your custom rule (Terminal+ view):mkdir node_modules npm install tslint@4.3.1
You will also need to fix require calls to refer to tslint module. I hope this will get you going until I provide you with complete solution.
Best regards,
Piotr Tomiak
gorandParticipantHellow!
I copy path: c:\eclipse\plugins\com.genuitec.eclipse.typescript_2.2.2.201703311443\bin\node_modules
to my path: \TSLint\node_modulesTSLint .\node_modules .\rules\andrTypedefRule.js
and modify require in file andrTypedefRule.js:
require(“../index”) => require(“../node_modules/tslint/lib/index”) require(“../language/utils”) => require(“../node_modules/tslint/lib/language/utils”)
It worked!
But I would like to do without copying all the files to my folder, but simply refer to the standard library with which WebClipse is currently running. Since every time you update WebClipse, it changes its location folder /node_modules/
For example, like this:
require(“%path_tslint%/index”)
require(“%path_tslint%/language/utils”)
support-piotrParticipantGorand,
I will include fix for your issue in the next release – CI 5. I will tweak node.js to prepend webclipse installation (or the projects dir) to the include path, when dealing with custom rules.
Anyway, you should change your requires to not use relative paths, but absolute (I’ve missed the “lib” dir in my reply before):
require(“../index”) => require(“tslint/lib/index”) require(“../language/utils”) => require(“tslint/lib/language/utils”)
That way it won’t matter where in the folder hierarchy “node_modules” are located and will also be able to work once my fix is out.
- This reply was modified 7 years, 8 months ago by support-piotr.
gorandParticipantHellow!
How do I know if the update is installed?
Today, WebClipse has been updated (com.genuitec.eclipse.typescript_2.2.2.201704061509), but still the relative path does not work.
Maybe this update does not yet contain your changes?
Here’s what I did:
1) I deleted the node_modules folder from my folder containing my rules
2) indicated the relative path in require
3) restart eclipse – error path not foundAttachments:
You must be logged in to view attached files.
support-piotrParticipantGorand,
yes, this was CI4a release with several hot fixes. I couldn’t include fix for your problem as it requires extensive testing in TSLint area to ensure we don’t get regressions. The fix will be present in CI 5, which is due in 3-4 weeks. However, I will most likely be able to provide you with patched plugin once I have the fix in place, so that you don’t have to wait so long for it. It should be done early next week.
Best regards,
Piotr Tomiak
support-tonyKeymasterGorand,
This has now been fixed in our latest release, CI 5. Please go to Help->Webclipse->Check for Updates and follow the wizard to update to CI 5. Please let us know if you see any problems with the new release.
gorandParticipantHellow!
I modify my rulerequire(“../index”) => require(“tslint/lib/index”) require(“../language/utils”) => require(“tslint/lib/language/utils”)
There are a lot of mistakes in the new version CI5. Watch the my video: https://www.youtube.com/watch?v=9ng16R9iKdU
1) now TSLint problems are not shown in the code editor window (sometimes they appear, but an internal WebClipse error occurs and they disappear – see the video)
2) In the process of work there are too many internal errors WebClipse
3) Why are TSLint problems now displayed as errors and not as warnings? (as it was before)
The main question: now it is impossible to work properly with TypeScript and so tell me how to return to using CI4 version?
gorandParticipantHellow?
You see my video? https://www.youtube.com/watch?v=9ng16R9iKdU
How work into CI5 version ?
support-swapnaModeratorGorand,
Sorry that you are seeing issues with CI 5. Thank you for sharing the video.
We are investigating it further and we will get back to you soon.Apologies for inconvenience caused.
–Swapna
MyEclipse Support
support-swapnaModeratorGorand,
Can you please share with us the .log file located at workspace dir/.metadata/.log for further investigation?
–Swapna
MyEclipse Support
support-swapnaModeratorGorand,
Thank you for the log file. I have filed it for investigation with the dev team.
We will get back to you soon. Thank you for your patience.–Swapna
MyEclipse Support
gorandParticipantHellow!
Is the error corrected in the new version of CI5? Now it is absolutely impossible to work.
How do I get back to CI4 and turn off auto-update?
The situation is unacceptable when the development stops for the update!
See today log.zipAttachments:
You must be logged in to view attached files.
Wojciech GalanciakParticipantHi Gorad,
Thank you for reporting this issue. We are working on it and will try to provide a fix as soon as possible. As it is a blocker for you, we will soon provide you instructions about reverting to CI 4.
Kind regards,
Wojciech Galanciak
timwebbKeymasterGorad,
We’ve temporarily disabled evergreen server-side while we investigate this issue. From the log, it looks like you are using Webclipse as a plugin on Eclipse for Java Developers. Here is the update site to the CI 4a plug-in:
http://downloads.genuitec.com/downloads/products/webclipse/updates/wc2017ci4a
If you need the Angular IDE standalone distribution, please let us know. We will also look into a setting to disable Evergreen in our CI 6 update.
Thanks and again, thanks for your patience!
Tim
-
AuthorPosts