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

added example of ignoring a specific warning #5614

Closed
wants to merge 1 commit into from
Closed

added example of ignoring a specific warning #5614

wants to merge 1 commit into from

Conversation

MartinoMensio
Copy link
Contributor

Description

This PR includes an example for ignoring a specific warning that needs to be done differently from version 2.2 (breaking change). Since I never used the warnings package before and probably many spaCy users too, I think that an example is needed.
I also hope that this is the most appropriate way of ignoring one of the warnings declared and used by spaCy.

Types of change

It is a change in the documentation, adding a code example

Checklist

  • I have submitted the spaCy Contributor Agreement.
  • I ran the tests, and all new and existing tests passed.
  • My changes don't require a change to the documentation, or if they do, I've added all required information.

@adrianeboyd adrianeboyd added the docs Documentation and website label Jun 22, 2020
@adrianeboyd
Copy link
Contributor

Thanks, this is a helpful addition! I'd like to modify it slightly, but don't want to edit it here since it would push to your master branch (in general it's good idea to use a branch for PRs), so I'll create a new PR instead.

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

Successfully merging this pull request may close these issues.

None yet

2 participants