Skip to content

Remove captcha on examples#103

Merged
ryvasquez merged 2 commits into
masterfrom
166052278-remove-captcha-validation
May 17, 2019
Merged

Remove captcha on examples#103
ryvasquez merged 2 commits into
masterfrom
166052278-remove-captcha-validation

Conversation

@ryvasquez

Copy link
Copy Markdown
Contributor

Description

  • Remove captcha examples, it will be no longer need when rejecting a job

perseusl
perseusl previously approved these changes May 16, 2019

@perseusl perseusl left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Just the outdated branch 👍

josephniel
josephniel previously approved these changes May 17, 2019
narusemotoki
narusemotoki previously approved these changes May 17, 2019

@narusemotoki narusemotoki left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
You don't bump version number because we don't need to release?

Comment thread examples/updateTranslationJob.py
@ryvasquez
ryvasquez dismissed stale reviews from narusemotoki, josephniel, and perseusl via e6e4a93 May 17, 2019 03:47
@ryvasquez

Copy link
Copy Markdown
Contributor Author

@jeanklingler

Copy link
Copy Markdown

you should probably rebase and double check the CI 🙂

@ryvasquez

Copy link
Copy Markdown
Contributor Author

@jeanklingler sure I will rebase, however the error in CI only occurring in python3.3

"pyflakes" failed during execution due to "'str' object has no attribute 'arg'"
Run flake8 with greater verbosity to see more details

@ryvasquez
ryvasquez force-pushed the 166052278-remove-captcha-validation branch from e6e4a93 to c788771 Compare May 17, 2019 04:36
@ryvasquez
ryvasquez force-pushed the 166052278-remove-captcha-validation branch from c788771 to c7950d1 Compare May 17, 2019 06:31
@ryvasquez
ryvasquez marked this pull request as ready for review May 17, 2019 06:33

@narusemotoki narusemotoki left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gengo.py is changed. Good to change the version number.

@ryvasquez

ryvasquez commented May 17, 2019

Copy link
Copy Markdown
Contributor Author

@narusemotoki im not sure if its necessary, we dont change any implementation of behavior. maybe its not needed for this change. also I will release the 1.1.0 after this PR is merged from this PR #104

@ryvasquez
ryvasquez merged commit ea145d3 into master May 17, 2019
@ryvasquez
ryvasquez deleted the 166052278-remove-captcha-validation branch May 17, 2019 06:51
@narusemotoki

narusemotoki commented May 17, 2019

Copy link
Copy Markdown
Contributor

But otherwise we can't upload this library to PyPI because version is conflicting.

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

Successfully merging this pull request may close these issues.

6 participants