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

[Bug]: it should error when there is no main config file #25814

Closed
vanessayuenn opened this issue Jan 30, 2024 · 1 comment · Fixed by #26035
Closed

[Bug]: it should error when there is no main config file #25814

vanessayuenn opened this issue Jan 30, 2024 · 1 comment · Fixed by #26035
Assignees
Milestone

Comments

@vanessayuenn
Copy link
Contributor

Describe the bug

Project: SvelteUI
Version: 8.0.0-alpha.13
Bug/feedback:

The upgrade command didn't warn/error about not being able to find the main configuration file. The configuration is in a non-standard directory which the user must define with the --config-dir flag, but there was no prompt for this. This caused all automigrations to be skipped.

see #25692

To Reproduce

No response

System

No response

Additional context

No response

@ndelangen
Copy link
Member

It does actually warn the user, but I chatted with @shilman about this, and we decided to promote it to an error.

Here's what it does today:
Screenshot 2024-02-16 at 14 36 15

I'll fix this inside of #26035

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants