Skip to content

Bulk Diff reports don't properly support override filenames with a leading period #24

@OdatNurd

Description

@OdatNurd

Summary

While in a Bulk Diff report (either single or multiple package), any override that starts with a leading period (e.g. .gitignore) is not correctly recognized as an override filename. It lacks the proper syntax scope, which causes it to not syntax highlight correctly and not support the context menu items for overrides.

Expected Behaviour

A with this name should still be detected as an override, even if it is unlikely that it would be used. For example, PackageResourceViewer seems to hide such files from the list of overrides.

Actual Behaviour

The filename in question (.gitignore as well as .github/ISSUE_TEMPLATE.md in this case) was not correctly syntax highlighted.

How to Reproduce

[Steps to follow to reproduce the issue. Feature Requests can omit this if desired.]

  1. Use Preferences > Browse Packages... from the menu
  2. Create the folder OverrideAudit
  3. Create the file .gitignore
  4. Create a bulk diff for the OverrideAudit package

Environment

  • Sublime Text build: 3126
  • OverrideAudit version: v1.0.1
  • Operating System: Windows/MacOS/Linux (all are affected)
  • Case Sensitive Filesystem: Both

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions