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 temporary data directories with the namng pattern data-XXX #709

Merged

Conversation

chrisvire
Copy link
Member

There are times when I need to compare data from different apps or data generated by different version of SAB/DAB. So I have been copying the data to something like "data-sab", "data-12.1.1", etc.

I don't want these showing up in the changed files.`

Since we already had data-sandbox as a source directory, I had to do a bit of fudging with the patterns so that new files in that directory would not be ignored, but the data-sandbox/node_modules would still be ignored. Ugh.

There are times when I need to compare data from different apps or
data generated by different version of SAB/DAB.  So I have been copying
the data to something like "data-sab", "data-12.1.1", etc.

I don't want these showing up in the changed files.`

Since we already had data-sandbox as a source directory, I had to do
a bit of fudging with the patterns so that new files in that directory
would not be ignored, but the data-sandbox/node_modules would still be
ignored. Ugh.
Copy link
Contributor

@davidmoore1 davidmoore1 left a comment

Choose a reason for hiding this comment

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

Interesting

@chrisvire chrisvire merged commit b8806b1 into sillsdev:main Oct 30, 2024
1 check passed
@chrisvire chrisvire deleted the feature/ignore-temp-data-directories branch October 30, 2024 15:47
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