-
-
Notifications
You must be signed in to change notification settings - Fork 626
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Thanks CodeFirst for your generation contribution to Aurelia!
- Loading branch information
1 parent
c4d658f
commit 01c0272
Showing
2 changed files
with
1 addition
and
5 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
01c0272
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why have you removed the export for LogManager?
This has caused my build to failed with error - "error TS2305: Module ''aurelia-framework'' has no exported member 'LogManager'."
can you revert it please? or supply workaround to get the LogManger back again ?
01c0272
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apologies. I'll add it back. It's a tricky spot because the d.ts gnerator doesn't actually generate this, so we add it by hand for each release. Then, TSDoc, which we use to generate the API docs, gets confused by this and generates bogus data, so we have to remove it before generating docs. Basically, two separate tools both error in different ways depending on whether it's there or not. We wouldn't release without this though. I'll push the fix to master. It sounds like you are working from the master branch instead of an official release. I'd definitely recommend not doing that if you can.
01c0272
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for adding it back, i will change our code to work with specific branch.