Skip to content

fix: treat log files as plaintext#8599

Open
SSDWGG wants to merge 1 commit into
nextcloud:mainfrom
SSDWGG:fix/log-files-plaintext
Open

fix: treat log files as plaintext#8599
SSDWGG wants to merge 1 commit into
nextcloud:mainfrom
SSDWGG:fix/log-files-plaintext

Conversation

@SSDWGG
Copy link
Copy Markdown

@SSDWGG SSDWGG commented May 16, 2026

Summary

  • map .log files to plaintext highlighting
  • add a regression test for the extension mapping

Fixes #5253

Tests

  • npx vitest run src/tests/helpers/mappings.spec.ts

@mejo-
Copy link
Copy Markdown
Member

mejo- commented May 18, 2026

@SSDWGG thanks as well for this contribution. As far as I can see, Text doesn't open .log files at all per default. So I wonder whether it's useful to add syntax highlighting conditions for it. Can you explain your usecase a bit?

Regarding the unit test, I think it can be dropped in any case. It doesn't test much except that the object key you introduced to mappings.ts has the value your put there.

@mejo- mejo- added enhancement New feature or request needs info labels May 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request needs info

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Color markings for .log files, not for .txt files

2 participants