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 .gitignore for standard Python #29

Open
wmwv opened this issue Sep 17, 2024 · 2 comments
Open

Add .gitignore for standard Python #29

wmwv opened this issue Sep 17, 2024 · 2 comments

Comments

@wmwv
Copy link
Collaborator

wmwv commented Sep 17, 2024

Add .gitignore for standard Python

Note: think carefully about FITS files. We do want some input, but not output. So perhaps include FITS (.fits, .fits.gz) in .gitignore and if anyone makes a change they'll have to remember to force the update.

@benjaminrose @sosey How do you feel about this list:
https://github.com/github/gitignore/blob/main/Python.gitignore

@benjaminrose
Copy link

I usually start with that repo and adjust as needed.

FITS: We can have predefined input names so we can add something like

*fits*
!input-*.fits

@sosey
Copy link

sosey commented Sep 17, 2024

I would also add to ignore *.asdf files. We dont want to be committing data files, or very large files, unless there is an exceeding good reason, and even then there are limitations in github.

we can also assign a reviewer for any commits that include a change to .gitignore through a codeowners file

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

No branches or pull requests

3 participants