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

Remove SQL_901 diagnostic hint with Update 4 dependency #3609

Closed
niveathika opened this issue Nov 4, 2022 · 3 comments
Closed

Remove SQL_901 diagnostic hint with Update 4 dependency #3609

niveathika opened this issue Nov 4, 2022 · 3 comments

Comments

@niveathika
Copy link
Contributor

Description:
Currently, the following HINT is added to the ERROR scenario.

_ = check charInOut.get();
ERROR: cannot infer type for parameter ''typeDesc''
HINT: parameter 'typeDesc' should be explicitly passed when the return data is ignored

With the Update 4 the error message is improved so that the message is more clear,

_ = check charInOut.get();
ERROR: cannot infer the ''typedesc'' argument for parameter ''typedesc'': expected an argument for the parameter or a contextually-expected type to infer the argument

With the improved error message the HINT is now obsolete and can be removed.

@anupama-pathirage
Copy link
Contributor

Can we close the issue since all PRs are merged? This issue is moved to done column without closing the issue @niveathika

@niveathika
Copy link
Contributor Author

We cannot merge the changes to the master branch, due to a constraint in stdlib release workflow. There is no way to configure the branch the release should be done.

Due to that reason this changes are kept in an upstream branch till the Update 3 releases are done. Since only syncing the change is remaining, I have moved this to done state. I will close this once the changes are merged to master

@niveathika
Copy link
Contributor Author

Closing this as all changes are synced to master

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

No branches or pull requests

2 participants