Fix: improve error message when a lib needs to be upgraded

[georgesittas]

No description provided.

[georgesittas]

Just lowercased the lib to avoid displaying "SQLGlot" in the pip command. For example (before vs after):

➜  foo sqlmesh plan
======================================================================
Successfully Ran 1 tests against duckdb
----------------------------------------------------------------------
Error: SQLGlot (local) is using version '18.10.2.dev14' which is behind '18.11.6' (remote). Please upgrade SQLGlot ('pip install SQLGlot --upgrade' command).
➜  foo sqlmesh plan
======================================================================
Successfully Ran 1 tests against duckdb
----------------------------------------------------------------------
Error: SQLGlot (local) is using version '18.10.2.dev14' which is behind '18.11.6' (remote). Please upgrade SQLGlot ('pip install sqlglot --upgrade' command).

[eakmanrq]

@georgesittas What do you think about providing the full command so they can just copy/paste?

[georgesittas]

@georgesittas What do you think about providing the full command so they can just copy/paste?

Which command do you mean? I think pip install sqlglot --upgrade suffices here for example right? (see example above)

Btw re: formatting, my intent here was also to copy the error message right above this one:

Please run a migration ('sqlmesh migrate' command)."

[eakmanrq]

Well it doens't include the version in it. So if they just copy/paste that and SQLGlot is ahead then it will install the wrong version.

[georgesittas]

Well it doens't include the version in it. So if they just copy/paste that and SQLGlot is ahead then it will install the wrong version.

Ah my bad, good catch. I'll fix this, thanks!

[georgesittas]

Updated message:

➜  foo sqlmesh plan
======================================================================
Successfully Ran 1 tests against duckdb
----------------------------------------------------------------------
Error: SQLGlot (local) is using version '18.10.1' which is behind '18.12.0' (remote). Please upgrade SQLGlot ('pip install "sqlglot==18.12.0"' command).

[eakmanrq]

@georgesittas Don't you need --upgrade included?

[georgesittas]

I thought the == forces the installation of that specific version? My impression was that --upgrade is required if we want to force-update even though the currently-used version is compatible with what we have specified in the project's package dependencies. Am I missing something?

EDIT: updated to include --upgrade anyway just to be sure.

[eakmanrq]

@georgesittas Ah you might be right. I just always do it. As you said it shouldn't hurt though.