Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

"verbose" argument to alembic.history cli command #40

Merged
merged 1 commit into from
May 26, 2022
Merged

"verbose" argument to alembic.history cli command #40

merged 1 commit into from
May 26, 2022

Conversation

krnr
Copy link
Contributor

@krnr krnr commented May 23, 2022

without this parameter command actually doesn't work:

 python manage.py db history
Traceback (most recent call last):
  File "manage.py", line 41, in <module>
    cli()
  File "/home/krnr/.pyenv/versions/ard38/lib/python3.8/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/home/krnr/.pyenv/versions/ard38/lib/python3.8/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/home/krnr/.pyenv/versions/ard38/lib/python3.8/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/krnr/.pyenv/versions/ard38/lib/python3.8/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/krnr/.pyenv/versions/ard38/lib/python3.8/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/krnr/.pyenv/versions/ard38/lib/python3.8/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/home/krnr/.pyenv/versions/ard38/lib/python3.8/site-packages/sqla_wrapper/cli/click_cli.py", line 71, in history
    alembic.history(verbose, start=start, end=end)
TypeError: history() takes 1 positional argument but 2 positional arguments (and 2 keyword-only arguments) were given

@jpsca
Copy link
Owner

jpsca commented May 26, 2022

I missed that, thank you

@jpsca jpsca merged commit 049d150 into jpsca:main May 26, 2022
@jpsca
Copy link
Owner

jpsca commented May 26, 2022

I just did a new release (v5.6.1) with the fix

@krnr krnr deleted the patch-1 branch June 4, 2022 08:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants