Skip to content

Add more temporary file patterns to .gitignore#10288

Merged
joschi merged 2 commits intodropwizard:release/4.0.xfrom
rhowe:gitignore
Aug 19, 2025
Merged

Add more temporary file patterns to .gitignore#10288
joschi merged 2 commits intodropwizard:release/4.0.xfrom
rhowe:gitignore

Conversation

@rhowe
Copy link
Copy Markdown
Contributor

@rhowe rhowe commented Jul 24, 2025

No description provided.

@rhowe rhowe requested a review from a team as a code owner July 24, 2025 05:59
@github-actions github-actions bot added this to the 4.0.15 milestone Jul 24, 2025
@joschi joschi modified the milestones: 4.0.15, 4.0.16 Jul 24, 2025
.gitignore Outdated
Comment on lines +23 to +25
# vim
*.swp

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be handled in a global git-ignore file.

Suggested change
# vim
*.swp

.gitignore Outdated
Comment on lines +1 to +3
# Git
*.orig

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure when this can happen. 🤔

Suggested change
# Git
*.orig

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's git mergetool detritus but global ignore works

.gitignore Outdated
Comment on lines +38 to +39
# Test files
/dropwizard-migrations/src/test/resources/test-db.mv.db
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please move this into a .gitignore file inside dropwizard-migrations?

Copy link
Copy Markdown
Member

@joschi joschi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rhowe Thanks!

@joschi joschi merged commit 6a3cd4b into dropwizard:release/4.0.x Aug 19, 2025
10 of 11 checks passed
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.

2 participants