Vampire
08/15/2020, 8:29 AM@actions/cache
to my npm dependencies.
Now I get 2285 compile errors because Dukat is creating many duplicated declarations. 😕
Anyone seen this before and has some idea?turansky
08/15/2020, 9:43 AM@actions
:
1. Run dukat
2. Fix errors
3. Add kotlin package
Also you can ask Vladimir to publish declarations to simplify declaration sharing :)Vampire
08/16/2020, 10:06 AMdoLast
action that fixes problems in the generated files.
But 2286 errors is quite a bit, and many are duplicate declarations in node.is files. :-/vladimirsitnikov
08/16/2020, 5:07 PMAdditionally I have aOh, that would probably take a while. Frankly speaking, I would love to go with that approach (all headers generated by Dukat) as well. Here are the issues that bother me the most: 1. All headers are generated into the root package. That makes API uses awkward. For instanceaction that fixes problems in the generated files.doLast
(@actions/core).info
should probably be converted to package actions.core
2. Overloads are not generated properly (Dukat often produces conflicting overloads)
3. Other Dukat issues 🙂
However, I would try to implement the business logic first, and only then go with fixing Dukat.Vampire
08/16/2020, 8:32 PMAll headers are generated into the root package.Yes, that's indeed annoying.
Overloads are not generated properly (Dukat often produces conflicting overloads)
Other Dukat issuesYep, also quite annoying, but at least so far I was able do work-around these in the
doLast
with not too much effort.
But that I now get 2286 is really bad.
There are many redeclarations, conflicting declarations and duplications now, even in ....module_node.kt
files, since I added @actions/cache
.
And I currently see no real way besides also not using the built-in integrated Dukat task. 😕
And this is actually the last damn dependency I had to add. 😞Vampire
08/17/2020, 9:39 AMVampire
08/19/2020, 1:03 PM