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

Fix gitignore for template files in templates folder #144

Merged
merged 2 commits into from
Aug 18, 2020

Conversation

naamor
Copy link
Member

@naamor naamor commented Aug 10, 2020

No description provided.

@lbovet
Copy link
Member

lbovet commented Aug 17, 2020

What is the effect of the proposed change?
You exclude templates/ from ignores. But it should already be the case, no? Everything that is not ignored is taken by default.

@naamor
Copy link
Member Author

naamor commented Aug 18, 2020

Because of the "gen" in generated gitignore and the "gen" in template path "root_directory/templates/plugin_name/gen/plugin_name/filename.ext" the template files are ignored too.

@lbovet
Copy link
Member

lbovet commented Aug 18, 2020

Oh, I see now. We could also have "/gen/" in .gitignore to ignore it only if it is on the root. Probably better.

@naamor
Copy link
Member Author

naamor commented Aug 18, 2020

Thanks for your feedback. I like your solution. Made a new commit

@lbovet lbovet merged commit 71a483a into swisspost:develop Aug 18, 2020
@lbovet
Copy link
Member

lbovet commented Aug 18, 2020

Thx!

@lbovet lbovet mentioned this pull request Aug 26, 2020
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