Skip to content

Conversation

@joshuaellis
Copy link
Contributor

What does it do?

  • adds the runtime directory for strapi apps to the generated gitignore

Why is it needed?

  • nicer DX – they don't need to commit this

Related issue(s)/PR(s)

  • raised on slack

@joshuaellis joshuaellis added pr: chore This PR contains chore tasks (cleanups, configs, tooling...) source: generators:app labels Dec 5, 2023
@joshuaellis joshuaellis requested a review from Marc-Roig December 5, 2023 08:20
@joshuaellis joshuaellis self-assigned this Dec 5, 2023
@derrickmehaffy
Copy link
Member

We may want to keep the .cache in there just in case (won't hurt anything).

dgrebb added a commit to dgrebb/dgrebb.com that referenced this pull request Dec 6, 2023
dgrebb added a commit to dgrebb/dgrebb.com that referenced this pull request Dec 6, 2023
@joshuaellis
Copy link
Contributor Author

We may want to keep the .cache in there just in case (won't hurt anything).

I think on this occasion because it's there's already so many items in that list and this is for new projects, we can probably just leave it because it shouldn't exist. The only case I can think of is if someone upgrades and we mutate the .gitignore, but i don't think thats in scope of the tools 👀

@joshuaellis joshuaellis merged commit 5582657 into develop Dec 12, 2023
@joshuaellis joshuaellis deleted the chore/add-runtime-dir-to-generator-gitignore branch December 12, 2023 12:52
dgrebb added a commit to dgrebb/dgrebb.com that referenced this pull request Dec 21, 2023
@strapi-bot
Copy link

This pull request has been mentioned on Strapi Community Forum. There might be relevant details there:

https://forum.strapi.io/t/auto-generated-files-to-git-or-not-to-git/35023/6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: chore This PR contains chore tasks (cleanups, configs, tooling...)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants