Skip to content

Update Godot.gitignore#4801

Merged
dooleydevin merged 1 commit into
github:mainfrom
NoahGreer:update-godot-gitignore-template
May 18, 2026
Merged

Update Godot.gitignore#4801
dooleydevin merged 1 commit into
github:mainfrom
NoahGreer:update-godot-gitignore-template

Conversation

@NoahGreer
Copy link
Copy Markdown
Contributor

@NoahGreer NoahGreer commented Feb 21, 2026

Link to the application or project's homepage

https://godotengine.org/

Reasons for making this change

As of Godot 4.2 the editor has a Windows-specific workaround for loading extension libraries. Before the extension library is loaded the Godot editor creates a temp copy of the extension library .dll file with a ~ prefix (e.g. example.dll => ~example.dll). When the extension library is unloaded the Godot editor deletes the temp copy. These copied ~*.dll files are temporary and should be ignored.

Links to documentation supporting these rule changes

Godot 4.2 changelog
https://godotengine.org/article/godot-4-2-arrives-in-style/#gdextension

Godot PR #80188 GDExtension: Copy DLL to a temp file before opening
godotengine/godot#80188

Merge and Approval Steps

@NoahGreer NoahGreer requested a review from a team as a code owner February 21, 2026 04:01
@doriangurazdowski0-prog
Copy link
Copy Markdown

@NoahGreer
Copy link
Copy Markdown
Contributor Author

NoahGreer commented Mar 23, 2026

Is there anything more I can do to help get this PR ready to be reviewed and merged? I read the contribution guidelines and believe I have followed them correctly. The only thing that remains unclear is how I am expected to complete the step "Ensure CI is passing". The action runner seems to be restricted to repo maintainers, so I presume that this step is something that can only be done by a repo maintainer.

Edit: the PR template was updated remove the bit about ensuring CI is passing, so I updated the description of this PR to match the latest template.

As of Godot 4.2 the editor has a Windows-specific workaround for loading extension libraries.
Before the extension library is loaded the Godot editor creates a temp copy of the extension library `.dll` file with a `~` prefix (e.g. `example.dll` => `~example.dll`).
When the extension library is unloaded the Godot editor deletes the temp copy.
These copied `~*.dll` files are temporary and should be ignored.

References:

Godot 4.2 changelog
https://godotengine.org/article/godot-4-2-arrives-in-style/#gdextension

Godot PR #80188 GDExtension: Copy DLL to a temp file before opening
godotengine/godot#80188
@NoahGreer NoahGreer force-pushed the update-godot-gitignore-template branch from 8cd0f4e to bf4440e Compare March 23, 2026 17:13
@NoahGreer NoahGreer changed the title Add rule for Godot temp extension library copies on Windows Update Godot.gitignore May 17, 2026
@NoahGreer
Copy link
Copy Markdown
Contributor Author

NoahGreer commented May 17, 2026

Hi @dooleydevin, I noticed you had reviewed and merged more recent PRs that were created after this one.
I don't want to come across as rude for @ing you directly, but there does not appear to be a way for me to request review, and this PR has been open for several months, so I'm @ing you as a proxy for requesting review.

Also, I noticed that since I initially opened this PR the pull request template for this repo was updated in #4843, so I revised this PR's description to match the current template.

@dooleydevin
Copy link
Copy Markdown
Collaborator

No worries @NoahGreer 😄 it's just taking us a long time to get in control of this repo's backlog. This looks good to me!

@dooleydevin dooleydevin merged commit 3780fff into github:main May 18, 2026
@NoahGreer NoahGreer deleted the update-godot-gitignore-template branch May 18, 2026 16:40
@NoahGreer
Copy link
Copy Markdown
Contributor Author

No worries @NoahGreer 😄 it's just taking us a long time to get in control of this repo's backlog. This looks good to me!

Thank you!

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.

3 participants