Skip to content

ignore interface imports#440

Closed
endel wants to merge 1 commit intoTypeScriptToLua:masterfrom
endel:patch-1
Closed

ignore interface imports#440
endel wants to merge 1 commit intoTypeScriptToLua:masterfrom
endel:patch-1

Conversation

@endel
Copy link
Copy Markdown
Contributor

@endel endel commented Feb 21, 2019

Hey there! I'm creating this pull-request as a draft, I haven't checked the whole source-code and what side effects this change may add. Let me know if you have any feedback, I'll gladly change it.

Cheers!

@endel
Copy link
Copy Markdown
Contributor Author

endel commented Feb 21, 2019

only one test failed with this:

not ok 29 Transformation Tests ( "exportStatement.ts", "exportStatement.lua" )
 ---
   message: "The test threw an unhandled error."
   severity: fail
   data:
     got: an unhandled error
     expect: no unhandled errors to be thrown
     stack: |
       TypeError: Cannot read property 'flags' of undefined
           at tryGetDeclaredTypeOfSymbol (/Users/endel/projects/TypeScriptToLua/node_modules/typescript/lib/typescript.js:36596:24)
           at Object.getDeclaredTypeOfSymbol (/Users/endel/projects/TypeScriptToLua/node_modules/typescript/lib/typescript.js:36593:20)
           at imports.elements.filter.e (/Users/endel/projects/TypeScriptToLua/src/LuaTransformer.js:210:51)
           at Array.filter (<anonymous>)
           at LuaTransformer.transformImportDeclaration (/Users/endel/projects/TypeScriptToLua/src/LuaTransformer.js:209:55)
           at LuaTransformer.transformExportDeclaration (/Users/endel/projects/TypeScriptToLua/src/LuaTransformer.js:173:39)
           at LuaTransformer.transformStatement (/Users/endel/projects/TypeScriptToLua/src/LuaTransformer.js:84:29)
           at statements.forEach.statement (/Users/endel/projects/TypeScriptToLua/src/LuaTransformer.js:140:83)
           at Array.forEach (<anonymous>)
           at LuaTransformer.transformStatements (/Users/endel/projects/TypeScriptToLua/src/LuaTransformer.js:139:20)
 ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant