Fix specific highlighting (CMakeLists.txt ...) #7686
No reviewers
Labels
No Label
backport/done
backport/v1.0
backport/v1.1
backport/v1.10
backport/v1.11
backport/v1.12
backport/v1.13
backport/v1.14
backport/v1.15
backport/v1.2
backport/v1.3
backport/v1.4
backport/v1.5
backport/v1.6
backport/v1.7
backport/v1.8
backport/v1.9
bounty
changelog
dependencies
frontport/done
frontport/main
good first issue
Hacktoberfest
hacktoberfest-accepted
in progress
kind/api
kind/breaking
kind/bug
kind/build
kind/deployment
kind/deprecated
kind/docs
kind/enhancement
kind/feature
kind/lint
kind/misc
kind/moderation
kind/package
kind/proposal
kind/question
kind/refactor
kind/regression
kind/security
kind/summary
kind/testing
kind/translation
kind/ui
kind/upstream-related
kind/usability
kind/ux
lgtm/done
lgtm/need 1
lgtm/need 2
performance/bigrepo
performance/cpu
performance/memory
performance/speed
priority/critical
priority/low
priority/maybe
priority/medium
proposal/rejected
reviewed/confirmed
reviewed/duplicate
reviewed/fixed
reviewed/invalid
reviewed/not-a-bug
reviewed/wontfix
skip-changelog
stale
status/blocked
status/needs-feedback
status/wip
theme/2fa
theme/authentication
theme/avatar
theme/backup-restore
theme/docker
theme/federation
theme/issues
theme/kanban
theme/markdown
theme/migration
theme/mobile
theme/pr
theme/signing
theme/sqlite
theme/timetracker
theme/webhook
theme/wiki
No Milestone
No project
No Assignees
1 Participants
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: lunny/gitea#7686
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "master"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
https://github.com/go-gitea/gitea/issues/7685
Add a new map for highlighting files that cannot be guessed with extension/filename (for instance, CMakeLists.txt).
Codecov Report
31.81% <0%> (ø)
Continue to review full report at Codecov.
Maybe we should move these after ignore check.
I don't think that it does change anything as they are exclusively different maps
Reviewing it I ask my self if we shouldn't replace highlightFileNames with map[string]string directly and only do one check.
Yes, seems like this is duplicating with the purpose
highlightFileNames
, so I'd say lowercase the name and put it there instead. I guess in most cases we do want case-insensitive filename matching.At least cmake requires CMakeFiles.txt file to be exactly like this named, so it is case-sensitive
I think this may also be true for
Dockerfile
andMakefile
actually, for which we currently do that case-insensitve check, so I guess those could be moved to sensitive checks too.I think that
make
command works also withGNUmakefile
andmakefile
filenames, whereascmake
only works withCMakeFiles.txt
.GNUmakefile
could be put in sensitive checks too.We can lower-case all. I don't think it would be too bad to highlight cmakefiles.txt like CMakeFiles.txt. Futhermore, I don't know how it work with case-insensitive fs.
Thanks for your first PR and your patience for the review process.