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

Gitignore numtracker.db #11

Merged
merged 1 commit into from
Dec 5, 2024
Merged

Gitignore numtracker.db #11

merged 1 commit into from
Dec 5, 2024

Conversation

callumforrester
Copy link
Contributor

Gitignore numtracker.db to make the quickstart process as simple as possible.
.gitignore has a section for demo database files et al. which does not include numtracker.db, generated by the CLI by default.

Copy link
Collaborator

@tpoliaw tpoliaw left a comment

Choose a reason for hiding this comment

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

Thanks, I have enough untracked files in the repo I got used to ignoring them.

.gitignore Outdated Show resolved Hide resolved
Gitignore *.db to make the quickstart process as simple as
possible.
.gitignore has a section for demo database files et al. which does not
include numtracker.db, generated by the CLI by default.
Copy link
Collaborator

@tpoliaw tpoliaw left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM.

Out of interest, how does having untracked files make the quickstart process more complicated?

@callumforrester
Copy link
Contributor Author

I suppose that's not quite the right phrasing, following the quickstart guide creates an untracked file in my repo, which I have to deal with at some point.

@callumforrester callumforrester merged commit 68b8372 into main Dec 5, 2024
3 checks passed
@callumforrester callumforrester deleted the gitignore-numtracker-db branch December 5, 2024 13:25
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