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

Ignore upgrade check if user hasn't published config yet #628

Merged
merged 2 commits into from
Mar 11, 2023
Merged

Ignore upgrade check if user hasn't published config yet #628

merged 2 commits into from
Mar 11, 2023

Conversation

sangnguyenplus
Copy link
Contributor

@sangnguyenplus sangnguyenplus commented Mar 7, 2023

It will throw error if there is no config file in config/scribe.php.

We should check if this file exists or not, if it's not exists, just ignore upgrade config.

Before:
Screenshot 2023-03-07 at 15 42 57

After:
Screenshot 2023-03-07 at 15 45 05

@shalvah
Copy link
Contributor

shalvah commented Mar 10, 2023

Can you fix the tests?

@sangnguyenplus
Copy link
Contributor Author

@shalvah I have just fixed tests.

Please check again.

@shalvah shalvah changed the title Ignore upgrade check if user haven't published config yet Ignore upgrade check if user hasn't published config yet Mar 11, 2023
@shalvah shalvah merged commit ed7b085 into knuckleswtf:master Mar 11, 2023
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