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

Example in README has outdated code #67

Open
ghost opened this issue Nov 4, 2019 · 3 comments
Open

Example in README has outdated code #67

ghost opened this issue Nov 4, 2019 · 3 comments

Comments

@ghost
Copy link

ghost commented Nov 4, 2019

Hi all,

Thank you for this fantastic library, it will be very useful for me!

I was experimenting with this library for a project of mine and noticed that the README has an out of date usage of marshmallow.Schema where the strict=True argument is passed.

In marshmallow > 3 there was a breaking change introduced as per the changelog: https://marshmallow.readthedocs.io/en/stable/changelog.html?highlight=strict#b7-2018-02-03

I'm happy to put up a PR to update the README if you need!

@thijstriemstra
Copy link

thijstriemstra commented Nov 4, 2019

had the same issue

Traceback (most recent call last):
  File "/usr/local/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/local/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/foo/__main__.py", line 6, in <module>
    from boatlife.main import main
  File "/foo/main.py", line 41, in <module>
    @request_schema(RequestSchema(strict=True))
TypeError: __init__() got an unexpected keyword argument 'strict'

maybe also useful to include in the readme:

npm install --save swagger-ui-dist
cp -R node_modules/swagger-ui-dist /path/to/your/static-dir/swagger

@ghost
Copy link
Author

ghost commented Nov 12, 2019

Hi there, is there any update on this?

@maximdanilchenko
Copy link
Owner

Hi! Yes, it would be cool to include a list of difference between marshmallow < 3 and marshmallow >= 3 into documentation (and I know that many people still use 2.x versions). @sad-boy-uoa you are welcome to put up a PR. I will appreciate it:)

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

No branches or pull requests

2 participants