Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add notepad++ backups to gitignore #3682

Merged
merged 2 commits into from
Oct 19, 2021
Merged

add notepad++ backups to gitignore #3682

merged 2 commits into from
Oct 19, 2021

Conversation

Zbizu
Copy link
Contributor

@Zbizu Zbizu commented Sep 22, 2021

Changes Proposed

I'm sick of seeing .bak files every time I type git status. I also want to use my icon without risk of pushing it by accident. The thing with icon is already settled so we can safely ignore it.

EPuncker
EPuncker previously approved these changes Sep 22, 2021
@marmichalski
Copy link
Contributor

Stop being sick and setup your git properly.

https://docs.github.com/en/get-started/getting-started-with-git/ignoring-files#configuring-ignored-files-for-all-repositories-on-your-computer

.gitignore Outdated
#############
## TFS / OT
#############
config.lua
theforgottenserver.ico
Copy link
Member

Choose a reason for hiding this comment

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

There is an icon actually checked in so this doesn't make sense.

@slawkens
Copy link
Contributor

slawkens commented Oct 9, 2021

.bak seems legit, would be useful for more people I guess.

.ico not, seems its already checked in repo as pointed by @DSpeichert

.gitignore Outdated Show resolved Hide resolved
Co-authored-by: Evil Puncker <EPuncker@users.noreply.github.com>
@Zbizu Zbizu changed the title add notpad++ backups and tfs icon to gitignore add notpad++ backups to gitignore Oct 19, 2021
@Zbizu Zbizu changed the title add notpad++ backups to gitignore add notepad++ backups to gitignore Oct 19, 2021
@EPuncker EPuncker merged commit a26e2e1 into otland:master Oct 19, 2021
@Zbizu Zbizu deleted the patch-4 branch November 16, 2021 22:38
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.

None yet

5 participants