deprecating legacy plot API

[edublancas]

once #508 is merged, we should start showing deprecation warnings:

jupysql/src/sql/run.py

Line 232 in 8ce3334

def pie(self, key_word_sep=" ", title=None, **kwargs):

jupysql/src/sql/run.py

Line 302 in 8ce3334

def bar(self, key_word_sep=" ", title=None, **kwargs):

and tell people they should use the %sqlplot API

[tl3119]

So, for this issue, we should not use the pie and bar function in run.py. Instead, we should use %sqlplot defined in plot.py. So we need to show a warning in the run.py to tell the user they should use the %sqlplot API?

[edublancas]

correct.

[edublancas]

@bryannho let's work on getting the PR merged. you have some issues in the CI so merge from master. i think that'll fix it

[bryannho]

got it, will create a new PR to merge from master

[edublancas]

I see you created a new PR, that's fine but for next time there's no need to do it, you can just update the existing PR and merge from master to that PR