-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
chore: prepend version mismatch to GraphQL errors #3580
base: master
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@petertonysmith94 Should we also remove the warning here?
if (/^\d+\.\d+\.\d+\D+/m.test(networkVersion)) { | |
// eslint-disable-next-line no-console | |
console.warn(`You're running against an unreleased fuel-core version: ${networkVersion}. Things may work as expected, but it's not guaranteed. Please use a released version. | |
This unreleased fuel-core build may include features and updates not yet supported by this version of the TS-SDK.`); | |
} |
@Torres-ssf I believe this is only a warning for when using unofficial versions of I'd say this one is alright to leave in. |
Coverage Report:
Changed Files:
|
Release notes
In this release, we:
Summary
console
) of incompatibilities with thefuel-core
node version on Provider instantiation. This could often lead to misinterpretation of issues.Checklist