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 a gitignore for build artifacts #66

Closed
lobre opened this issue Nov 24, 2023 · 3 comments · May be fixed by #70
Closed

Add a gitignore for build artifacts #66

lobre opened this issue Nov 24, 2023 · 3 comments · May be fixed by #70

Comments

@lobre
Copy link
Contributor

lobre commented Nov 24, 2023

Hello,

When building the project with make, it leaves a lot of build-related files untracked in git.

This would benefit from having a .gitignore to avoid listing them in git.

Would you be accepting a PR for that?

$ git status
On branch master
Your branch is up to date with 'origin/master'.

Untracked files:
  (use "git add <file>..." to include in what will be committed)
        cmd.o
        conf.o
        dir.o
        ex.o
        lbuf.o
        led.o
        mot.o
        reg.o
        regex.o
        ren.o
        rset.o
        rstr.o
        sbuf.o
        syn.o
        tag.o
        term.o
        uc.o
        vi
        vi.o

Thanks

@lobre
Copy link
Contributor Author

lobre commented Dec 20, 2023

I created a pull request on Github for this, but if I understand correctly, you are only interacting through emails. So here is the content of my PR here, in case you are interested in adding this .gitignore.

# libs
*.o
*.ko
*.obj
*.elf

# objects
*.dll
*.so
*.so.*
*.dylib

# executables
*.exe
*.out
*.hex
vi

@aligrudi
Copy link
Owner

aligrudi commented Dec 22, 2023 via email

@lobre
Copy link
Contributor Author

lobre commented Dec 26, 2023

Thank you, I will close this issue as your proposed gitignore is good enough for now.

@lobre lobre closed this as completed Dec 26, 2023
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 a pull request may close this issue.

2 participants