Skip to content
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

Serilog imports. #89

Closed
malkaviano opened this issue Jun 2, 2019 · 6 comments
Closed

Serilog imports. #89

malkaviano opened this issue Jun 2, 2019 · 6 comments
Labels

Comments

@malkaviano
Copy link

The documentation should document what are the Serilog most important namespaces.

Not everybody is using VS and VS Code does not have that feature (thank god, now I realize how stupid one can become relying too much on the IDE) .

Btw, I discovered the namespaces looking @ the source code, but you really should consider documenting it and updating the install package command.

@adamchester
Copy link
Member

Thanks for the suggestion @malkaviano. May I ask more specifically what you were doing that needed to open namespaces related to this package?

@malkaviano
Copy link
Author

@adamchester Like to run the sample on the README one need to know what namespaces to import.

Where else would I look for?

@nblumhardt
Copy link
Member

Is it just using Serilog; that your configuration needed, @malkaviano ? Scanning this README I can't spot any code that should need others, but I could be missing it....

@nblumhardt
Copy link
Member

(I'm keen to add using Serilog; in there to clarify it, but as I think @adamchester is getting at, we want to make sure we solve the actual problem that you're hitting :-))

@malkaviano
Copy link
Author

Yes, that will do the trick.

using Serilog.Events; is also nice for LogEventLevel.

@nblumhardt
Copy link
Member

Thanks @malkaviano - I've added using Serilog; - in this example I don't think LogEventLevel plays a very prominent part so we might be best off saving the clutter, for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants