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

UI: Create .gitignore file on postBuild #18045

Merged
merged 1 commit into from
Nov 18, 2022
Merged

Conversation

hashishaw
Copy link
Contributor

A .gitkeep file was added to http/web_ui in order to make the folder compliant with Go's native embedding (#14246). However, ever since then while developing on the UI the file would get deleted locally, which caused frustration and overhead for developers to remember not to commit those changes.

The root cause is that the contents of that folder are generated on hot reload whenever we run the ember server, and the folder contents get overwritten. This PR adds and in-repo addon which writes the .gitkeep file after build for all build types (development, production).

@hashishaw hashishaw added ui pr/no-changelog devex Developer Experience labels Nov 18, 2022
Copy link
Contributor

@zofskeez zofskeez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unreal. Utmost gratitude for taking care of this! 🙇

@hashishaw hashishaw merged commit 17bad82 into main Nov 18, 2022
@hashishaw hashishaw deleted the ui/keep-gitignore-v3 branch November 18, 2022 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devex Developer Experience pr/no-changelog ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants