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

Move priv direcotry gitignore from quickjs app to main gitignore #5076

Merged
merged 1 commit into from
Jun 4, 2024

Conversation

nickva
Copy link
Contributor

@nickva nickva commented Jun 3, 2024

Previously, the .gitignore file in priv was mainly there to keep the directory around in git. But that's a bit dirty as the debian packager will complain if we package the .gitignore, as it packages the whole priv directory.

To fix, move the .gitignore entries to the app's main .gitignore, and make sure to create and clean the whole priv directory as needed.

Previously, the `.gitignore` file in priv was mainly there to keep the
directory around in git. But that's a bit dirty as the debian packager will
complain if we package the `.gitignore`, as it packages the whole `priv`
directory.

To fix, move the .gitignore entries to the app's main .gitignore, and make sure
to create and clean the whole priv directory as needed.
@nickva nickva merged commit a145e12 into main Jun 4, 2024
23 checks passed
@nickva nickva deleted the remove-ignore-from-quickjs branch June 4, 2024 02:14
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.

None yet

2 participants