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

Add overwrite:false file config and better parser matching #323

Merged
merged 2 commits into from
Jul 7, 2023

Conversation

lukekarrys
Copy link
Contributor

@lukekarrys lukekarrys commented Jul 6, 2023

This PR contains two commits:

  1. Makes file parsers based on the target filename instead of the source filename. This better matches the expected behavior since the target filename is usually standard (like package.json or .gitignore) but the source often needs to be named differently to avoid issues with those well known filenames.

  2. By default template-oss allows for written files to be configured on a per-repo basis. This is helpful for different repos to create their own templated files and apply those. With this change, a repo can now set overwrite: false to a templated file and have those updates be applied after the default template-oss changes are made.

@lukekarrys lukekarrys requested a review from a team as a code owner July 6, 2023 00:29
@lukekarrys lukekarrys force-pushed the lk/stack-changed-files branch 11 times, most recently from 12e70a8 to 51d5f7c Compare July 6, 2023 06:29
@lukekarrys lukekarrys changed the title feat: add overwrite: false property to added files Add overwrite:false file config and better parser matching Jul 6, 2023
@lukekarrys lukekarrys force-pushed the lk/stack-changed-files branch 3 times, most recently from be89175 to 4de68f5 Compare July 6, 2023 06:40
By default template-oss allows for written files to be configured on a
per-repo basis. This is helpful for different repos to create their own
templated files and apply those.

With this change, a repo can now set overwrite: false to a templated
file and have those updates be applied after the default template-oss
changes are made.
@lukekarrys lukekarrys merged commit 4624d9c into main Jul 7, 2023
23 checks passed
@lukekarrys lukekarrys deleted the lk/stack-changed-files branch July 7, 2023 15:30
@github-actions github-actions bot mentioned this pull request Jul 7, 2023
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