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

Ignore all generated files #25

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

slyfox3
Copy link
Contributor

@slyfox3 slyfox3 commented Jun 6, 2021

Before this change, git status shows these untracked files:

Makefile
Makefile.in
aclocal.m4
autom4te.cache/
config.h
config.h.in
config.log
config.status
configure
data/Makefile
data/Makefile.in
ibus-array.spec
icons/Makefile
icons/Makefile.in
libtool
ltmain.sh
m4/Makefile
m4/Makefile.in
py-compile
setup/Makefile
setup/Makefile.in
setup/config.py
setup/ibus-setup-array
src/.deps/
src/Makefile
src/Makefile.in
src/array.inputMethod.xml.in
stamp-h1

Before this change, `git status` shows these untracked files:
`
Makefile
Makefile.in
aclocal.m4
autom4te.cache/
config.h
config.h.in
config.log
config.status
configure
data/Makefile
data/Makefile.in
ibus-array.spec
icons/Makefile
icons/Makefile.in
libtool
ltmain.sh
m4/Makefile
m4/Makefile.in
py-compile
setup/Makefile
setup/Makefile.in
setup/config.py
setup/ibus-setup-array
src/.deps/
src/Makefile
src/Makefile.in
src/array.inputMethod.xml.in
stamp-h1
`
@lexical
Copy link
Owner

lexical commented Jun 16, 2021

I think to get make clean to remove these files may make more sense. Don't you think?

@slyfox3
Copy link
Contributor Author

slyfox3 commented Jun 16, 2021

I think to get make clean to remove these files may make more sense. Don't you think?

Not really. It's common to ignore the built artifacts. In a typical normal dev-cycle we'd just edit-build-commit, and the untracked files are just too annoying to see, at least to me.

@slyfox3
Copy link
Contributor Author

slyfox3 commented Jun 25, 2021

@anthonyfok, thoughts?

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