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 wake build script. #2847

Merged
merged 1 commit into from
Jun 22, 2021
Merged

remove wake build script. #2847

merged 1 commit into from
Jun 22, 2021

Conversation

sequencer
Copy link
Member

Type of change: other enhancement
Impact: no functional change
Development Phase: implementation

@hcook hcook merged commit 85ffc5c into master Jun 22, 2021
@hcook hcook deleted the remove_wake branch June 22, 2021 19:19
@davidmlw
Copy link

What is the consideration of removing build.wake? It is a cool feature. @sequencer @jackkoenig

@sequencer
Copy link
Member Author

There is a big maintain burden, and leak of users, so we choose to remove it.
In the future, a better way is splitting RC and switching to ivy-based compiling strategy.

@davidmlw
Copy link

I see. Wish to see ivy- and mill-based flow. I would like to offer help on wake based scripts, when the RC is splitted.

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.

4 participants