-
Notifications
You must be signed in to change notification settings - Fork 833
docs: improve FAQ entry about disabling snapshot.auto-track #8267
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
base: main
Are you sure you want to change the base?
Conversation
josephlou5
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just checking this out because I was curious and saw a couple things, but probably need review from someone else who knows better 🙃
|
Doh! Just saw there already is an entry about it... Let's see if there is any value to merging the two. Otherwise I will just close this PR. |
84442ca to
7fc4da0
Compare
|
So, the existing entry focuses on "scratch files". Meaning, I presume, files that are never intended to be tracked & committed. Maybe there could be some value in saying something for users that are actually trying to get back to the git workflow. In that case, users are adding new files that may end up being committed later, they just want to do that tracking explicitly. And regarding the scratch files, it might be good to mention the workflow where people commit scratch files and squash it into a commit that lives parallel to any feature branches being worked on. A megamerge then brings all changes into the working copy. |
7fc4da0 to
5a26c08
Compare
|
if you want to see some previous plus and minus points for auto-tracking just take the time to scroll through #323. |
|
Yeah I started doing that and went back and forth with my ideas to improve the FAQ entry... I was a little trigger happy with opening the PR. |
CHANGELOG.mdREADME.md,docs/,demos/)cli/src/config-schema.json)