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

Remove dist folder from version control #328

Merged
merged 3 commits into from Feb 19, 2021
Merged

Conversation

silverbucket
Copy link
Member

@silverbucket silverbucket commented Feb 18, 2021

The dist folder used to have a purpose for being checked in to the repo, but that changed a while back and I just never removed this. So doing it now. The dist folder is now always generated at run time (ts->js) and is also used during package publishing.

@silverbucket silverbucket self-assigned this Feb 18, 2021
@silverbucket silverbucket changed the title Remove dist folder for version control Remove dist folder from version control Feb 18, 2021
@silverbucket
Copy link
Member Author

noice!
Screenshot 2021-02-18 at 23 38 30

Copy link
Contributor

@galfert galfert left a comment

Choose a reason for hiding this comment

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

👍

@silverbucket silverbucket merged commit 99d660b into master Feb 19, 2021
@silverbucket silverbucket deleted the td/remove-dist-folder branch February 19, 2021 11:36
silverbucket added a commit that referenced this pull request Mar 16, 2021
* remove and ignore dist folder
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants