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

Exclude unused files from download archives #140

Merged
merged 1 commit into from
Oct 13, 2023
Merged

Exclude unused files from download archives #140

merged 1 commit into from
Oct 13, 2023

Conversation

deathaxe
Copy link
Contributor

This PR proposes to...

  1. mark all files and folders export-ignore, which are not required to install the dependency.
  2. ignore ST project and workspace files to make sure to keep them local, if exist.

A minor improvement to make sure to download only what's needed.

The idea is to create mdpopups.sublime-package file with everything except st3 folder to keep package style resources (e.g.: Preferences.sublime-settings) available.

This commit...

1. marks all files and folders `export-ignore`, which are not required
   to install the dependency.
2. ignores ST project and workspace files to make sure to keep them local,
   if exist.
@gir-bot gir-bot added S: needs-review Needs to be reviewed and/or approved. C: infrastructure Related to project infrastructure. labels Oct 13, 2023
@facelessuser
Copy link
Owner

@gir-bot lgtm

@gir-bot gir-bot added S: approved The pull request is ready to be merged. and removed S: needs-review Needs to be reviewed and/or approved. labels Oct 13, 2023
@facelessuser
Copy link
Owner

Thanks, this is a useful PR!

@facelessuser facelessuser merged commit 388a1b0 into facelessuser:master Oct 13, 2023
6 checks passed
@deathaxe deathaxe deleted the pr/export-ignore branch October 13, 2023 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: infrastructure Related to project infrastructure. S: approved The pull request is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants