-
Notifications
You must be signed in to change notification settings - Fork 14
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
Logger #2
Comments
Let's keep things simple until there is a compelling reason to do otherwise. NLog should be fine for now. |
There is a little problem with NLog. Last night I have faced an issue related to NLog supported targets. Since I created a Portable Class Library Project, I can not add NLog. For now, I implemented logger with So, there are two option to create a library project:
So, I thought that Portable will be better but it came with some caveats. The first problem was Sorry for the confussion and taking your time, since I am not a .Net developer I couldn't know what problems I will face so time to time I will need to take your opinion. Btw, you can check difference between Class Library and Class Library Portable by MSDN, StackOverflow and a Blog. |
I suspect whatever we do will have to change when Windows 10 is released, which will be the main target of this code (both for phones and for desktop Windows 10), because this is the first platform with native beacon support. It is unclear whether Windows 10 will use Portable Class Libraries to allow code to work on both platforms. I see almost no reason to make code that will run on Windows 8.x mobile. I therefore think it is reasonable to start with a regular Class Library. Major caveat: My Windows dev skills are very rusty. My last C# app was 9 years ago. |
Okay, I will try to convert project type than. Btw, portable library also supports metro apps which makes me think about it. |
Hey @davidgyoung,
Like I mentioned in #1,
LogManager
is not implemented. I am planing to implement it first and NLog is different than Android Logger. Even, I think there is no need for an additional LogManager since NLog is configurable by its conf file. What do you think about it ? Should I do it in NLog way or same as Android ?The text was updated successfully, but these errors were encountered: