Skip to content
This repository has been archived by the owner on Dec 17, 2023. It is now read-only.

Add a fix mode that automatically fixes unused exports (revival) #104

Closed
wants to merge 12 commits into from

Conversation

phiresky
Copy link
Contributor

Revival of #67. Also fixes #98.

I added git status checking, if the git repo containing the given project has any uncommitted changes, it will refuse to modify it.

@phiresky
Copy link
Contributor Author

CI runs into out of memory for some reason, not sure why?

@github-actions
Copy link
Contributor

Stale pull request message

@nadeesha
Copy link
Owner

@phiresky Would you be able to merge master and try again? (Sorry about the delay in response)

@phiresky
Copy link
Contributor Author

It's still runnnig out of memory? :o i don't think this changes anything that should affect that?

@github-actions github-actions bot closed this Feb 5, 2021
@nadeesha nadeesha reopened this Feb 5, 2021
@github-actions github-actions bot closed this Feb 13, 2021
@nadeesha nadeesha reopened this Feb 14, 2021
@nadeesha
Copy link
Owner

I'm not really sure why this is the case. It might be that .forEach is called synchronously but decl.remove() is executed async, which makes too many file handles to be opened at the same time.

@mgwidmann
Copy link

would like to see this merged, is there an update?

@nadeesha
Copy link
Owner

Please open this once you have a change to clean this up. I'm closing this for now.

@nadeesha nadeesha closed this Aug 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] Add option for process.exit() with code greater than zero for build failure
3 participants