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

Cache gitignore.io tokens to avoid requests at build time #51

Merged
merged 10 commits into from
Jun 23, 2023

Conversation

NiklasRosenstein
Copy link
Contributor

@NiklasRosenstein NiklasRosenstein commented Jun 22, 2023

  • Reimplement GitignoreSyncTask
  • Use GitignoreSyncTask.create_check() inherited from RenderFileTask instead of custom sync task
  • Simplify GitignoreFile class, move logic to find and replace generated content sections into a separate module
  • Cache a subset of Gitignore.io tokens and distribute it as a GZip compressed JSON as part of kraken-std

@NiklasRosenstein NiklasRosenstein self-assigned this Jun 22, 2023
NiklasRosenstein and others added 7 commits June 22, 2023 23:26
…ib.check_file_contents_task` to `kraken.common.strings`
…f `Property.set()` (or its variants) have not been called before and if the property does not have a default value.
…ifying the code by a lot (e.g. no more tracking of a generated content hash) and cache a subset of tokens from gitignore.io to distribute them as part of kraken-std. The old begin/end markers we used in gitignore files before is still supported. We also no longer sort the gitignore file entries.
@NiklasRosenstein NiklasRosenstein marked this pull request as ready for review June 23, 2023 10:52
@NiklasRosenstein NiklasRosenstein merged commit 97ff38d into develop Jun 23, 2023
11 checks passed
@NiklasRosenstein NiklasRosenstein deleted the nr/cache-gitignore-io-tokens branch June 23, 2023 11: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.

None yet

1 participant