Skip to content

dev: add desktop.ini files to gitignore#7133

Merged
damocleas merged 3 commits intopagefaultgames:betafrom
Bertie690:desktop-ini
Mar 2, 2026
Merged

dev: add desktop.ini files to gitignore#7133
damocleas merged 3 commits intopagefaultgames:betafrom
Bertie690:desktop-ini

Conversation

@Bertie690
Copy link
Contributor

What are the changes the user will see?

N/A

Why am I making these changes?

Damo noticed that a desktop.ini file was getting tracked by Git.
This is a Windows-specific file created to track explorer.exe info, and is re-created each time the folder is opened.

What are the changes from a developer perspective?

Added desktop.ini (in all subdirectories) to the gitignore.

Removed the gitignore entries for assets into a submodule-specific gitignore file.

Checklist

  • The PR content is correctly formatted:
    • I'm using beta as my base branch
    • The current branch is not named beta, main or the name of another long-lived feature branch
    • I have provided a clear explanation of the changes within the PR description
    • The PR title matches the Conventional Commits format (as described in CONTRIBUTING.md)
  • The PR is self-contained and cannot be split into smaller PRs
  • There is no overlap with another open PR

Does this require any additions or changes to in-game assets? If so:

@Bertie690 Bertie690 requested a review from a team as a code owner March 2, 2026 19:25
@Bertie690 Bertie690 requested review from DayKev, Wlowscha and fabske0 March 2, 2026 19:25
@Madmadness65 Madmadness65 added the Development Changes designed to make development easier label Mar 2, 2026
Madmadness65
Madmadness65 previously approved these changes Mar 2, 2026
fabske0
fabske0 previously approved these changes Mar 2, 2026
@Bertie690 Bertie690 dismissed stale reviews from fabske0 and Madmadness65 via 9399d23 March 2, 2026 19:45
@damocleas damocleas merged commit 505bcff into pagefaultgames:beta Mar 2, 2026
9 checks passed
@Bertie690 Bertie690 deleted the desktop-ini branch March 2, 2026 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development Changes designed to make development easier

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants