-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: add SpellChecker #1
Conversation
ac27c6d
to
271e233
Compare
691d4e3
to
be1c46d
Compare
be1c46d
to
6a442f7
Compare
Co-authored-by: Nan Wang <nan.wang@jina.ai>
ad0625a
to
8eda16e
Compare
8eda16e
to
6876386
Compare
c7ae750
to
eb601cd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor things. Otherwise, it is good to merge
eb601cd
to
38f3dce
Compare
38f3dce
to
574b969
Compare
@nan-wang The dockerfile is required. Otherwise it seems hubble push fails https://github.com/jina-ai/executor-spellchecker/runs/3936376198?check_suite_focus=true#step:6:200 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM👍
Add initial PR for SpellChecker Executor
TODO