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

fix: elevate logs to .warn & check CFs for @sveltejs/kit dep #9

Merged
merged 6 commits into from
Feb 8, 2021

Conversation

jthegedus
Copy link
Owner

@jthegedus jthegedus commented Feb 8, 2021

Summary

  • elevate logging level:

    The core of this adapter is to output helpful logs to the user when their configuration does not meet the requirements of either a Cloud Function or Cloud Run deployment. Updates to SvelteKit in 1.0.0-next.27 put the builder.log.info or .minor logging functions behind the --verbose flag. Given console output is the core feature here, we will elevate this to .warn log level when noting a validation issue with the user's config.

    Resolves part of bug: issues deploying to Cloud Functions #6

  • add a check and subsequent log for the required Cloud Function dep @sveltejs/kit

  • remove GitHub Releases publish workflow in favour of standard-version and manual release via npm run release

  • format the code with npx xo --fix

@jthegedus jthegedus added the bug Something isn't working label Feb 8, 2021
@jthegedus jthegedus self-assigned this Feb 8, 2021
@jthegedus jthegedus changed the title fix: elevate validation logs to .warn & check CFs for @sveltejs/kit dep fix: elevate logs to .warn & check CFs for @sveltejs/kit dep Feb 8, 2021
@jthegedus jthegedus merged commit 6b35ee1 into main Feb 8, 2021
@jthegedus jthegedus deleted the fix/elevate-logging-suggestions branch February 8, 2021 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant