Skip to content

Update README.md #84

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

Merged
merged 1 commit into from
Nov 10, 2022
Merged

Update README.md #84

merged 1 commit into from
Nov 10, 2022

Conversation

jules-goose
Copy link
Contributor

edited the sample json because the old version didn't work (capitalization error)

edited the sample json because the old version didn't work (capitalization error)
@SimonCropp
Copy link
Contributor

@jules-goose thanks

@SimonCropp SimonCropp merged commit 673dca2 into serilog:dev Nov 10, 2022
@nblumhardt
Copy link
Member

The older version is correct, these are the argument names in the ByExcluding(string expression) function call. I'm not sure why the lowercase version would be failing for you, might need to dig in deeper, I'll revert this for now.

@nblumhardt
Copy link
Member

One possibility - are you using multiple settings files with overrides and running into some unexpected conflict between them? (E.g. appsettings.json and appsettings.Development.json)?

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.

3 participants