Skip to content

Report the number of escape characters formatted into themed sequence rendering#136

Merged
nblumhardt merged 2 commits intoserilog:devfrom
nblumhardt:fix-sequence-width-reporting
Nov 2, 2025
Merged

Report the number of escape characters formatted into themed sequence rendering#136
nblumhardt merged 2 commits intoserilog:devfrom
nblumhardt:fix-sequence-width-reporting

Conversation

@nblumhardt
Copy link
Member

No test hook currently available for this that I can see.

Simple bug, escape sequences within rendered sequences will count towards the output width, but shouldn't.

@nblumhardt nblumhardt marked this pull request as ready for review November 28, 2023 20:26
@nblumhardt nblumhardt merged commit 489be94 into serilog:dev Nov 2, 2025
1 check passed
@nblumhardt
Copy link
Member Author

Should have had this one in a while ago; needs a quick test but can probably go out as a patch fairly soon.

@nblumhardt nblumhardt mentioned this pull request Nov 2, 2025
This was referenced Nov 3, 2025
microsoft-github-policy-service bot pushed a commit to Azure/bicep that referenced this pull request Feb 23, 2026
Updated
[Serilog.Sinks.Console](https://github.com/serilog/serilog-sinks-console)
from 6.0.0 to 6.1.1.

<details>
<summary>Release notes</summary>

_Sourced from [Serilog.Sinks.Console's
releases](https://github.com/serilog/serilog-sinks-console/releases)._

## 6.1.1

## What's Changed
* Report the number of escape characters formatted into themed sequence
rendering by @​nblumhardt in
serilog/serilog-sinks-console#136


**Full Changelog**:
serilog/serilog-sinks-console@v6.1.0...v6.1.1

## 6.1.0

* #​165 - support for `{UtcTimestamp}` in output templates (@​ManuelRin)
 * #​172, #​173 - switch build to Serilog org standard (@​nblumhardt)
 

Commits viewable in [compare
view](serilog/serilog-sinks-console@v6.0.0...v6.1.1).
</details>

[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=Serilog.Sinks.Console&package-manager=nuget&previous-version=6.0.0&new-version=6.1.1)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>
###### Microsoft Reviewers: [Open in
CodeFlow](https://microsoft.github.io/open-pr/?codeflow=https://github.com/Azure/bicep/pull/19074)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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