Skip to content

Conversation

leseb
Copy link
Contributor

@leseb leseb commented Apr 19, 2024

Previously when running ilab init --non-interactive, if a config.yaml was present it would ask whether it should be overwritten or not.
Now if the non-interactive mode is enabled the config.yaml will be overwritten.

Resolves #907
Signed-off-by: Sébastien Han seb@redhat.com

@leseb leseb requested a review from abhi1092 as a code owner April 19, 2024 07:41
Previously when running `ilab init --non-interactive`, if a
`config.yaml` was present it would ask whether it should be overwritten
or not.
Now if the non-interactive mode is enabled the `config.yaml` will be
overwritten.

Fixes: instructlab#907
Signed-off-by: Sébastien Han <seb@redhat.com>
@alimaredia alimaredia requested review from alimaredia and removed request for abhi1092 April 24, 2024 06:44
Copy link
Contributor

@alimaredia alimaredia left a comment

Choose a reason for hiding this comment

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

Lgtm! Let me figure out if we want this PR in the 5/2 bugfix release or the release after that on5/30. If we want the PR in the 5/2 release we'll merge the PR ASAP, if not we'll hold off for a couple of weeks.

@alimaredia alimaredia merged commit 61c9701 into instructlab:main Apr 25, 2024
@leseb leseb deleted the fix-907 branch April 25, 2024 11:38
@anik120 anik120 added this to the Release - 5/2 milestone Apr 25, 2024
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.

ilab init --non-interactive asks interactive question if config.yaml already exists
4 participants