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

[nix] add "--interactive" as default flag for mill #777

Merged
merged 1 commit into from
Sep 11, 2024
Merged

Conversation

Avimitin
Copy link
Contributor

Using mill without "-i/--interactive" or "--no-server" will run a mill build server automatically. The build server doesn't inherit or update environment when user invoke mill at command-line, which is hard for developers to debug.

As the T1 project doesn't use mill build server for a long time, I decided to add the "-i" flag as default for mill. If someday someone wants to use the unwrapped version, they can use the .mill-unwrapped executable contains in the same package.

Using mill without "-i/--interactive" or "--no-server" will run a mill
build server automatically. The build server doesn't inherit or update
environment when user invoke mill at command-line, which is hard for
developers to debug.

As the T1 project doesn't use mill build server for a long time, I
decided to add the "-i" flag as default for mill. If someday someone
wants to use the unwrapped version, they can use the .mill-unwrapped
executable contains in the same package.

Signed-off-by: Avimitin <[email protected]>
@sequencer sequencer merged commit e6139f5 into master Sep 11, 2024
159 checks passed
@sequencer sequencer deleted the fxxk-bsp branch September 11, 2024 09:48
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