Regex Tweak #370
Open
Regex Tweak #370
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The old world weather API is no more. The replacement does not provide the local time in the JSON output. The solution is migrating the plugin to use the google apis which provides the timeZoneId, which is definitely enough for the `pytz` to generate a datetime object.
In the console the HTML tags look pretty uggly and it's better to eliminate them completely.
Fix worldtime plugin
…_backend strip HTML tags for shell backend
Feature/dockerized
* Tenzer-all-the-changes: Add Jeppe Toustrup to the AUTHORS list Bugfix: Avoid the use of `locals()` in the reminder plugin Bugfix: Don't report HC/BB state if the previous state wasn't known Pylint: W0612, W0613, remove unused variables Pylint: C1801, don't use `len(i) == 0` as a condition in if statements Pylint: W1201, let the logging functions do string interpolation Bugfix: Correct regex for will.plugins.friendly.mornin plugin Cleanup: Remove broken random_topic plugin Python 3: Run tests in Python 3 along with Python 2 Python 3: Use new way of representing octal numbers Python 3: Fix detection of plugin methods Python 3: Use new form of `except` lines Python 3: Make imports absolute and clean up unused ones Python 3: Use six to handle imports which have changed between 2 and 3 Python 3: Update dependencies on dnspython and natural to get PY3 support Python 3: print is a function, not a statement
…and spelling errors.
Feature/merges in py3
* feature/merges-in-py3: Friendlier location output
… of friendliness.
…cally handled (with explanations)
There was no check whether the given channel name, regular or default, existed. This caused the thread to fail and stall Will until it was restarted.
Fix fabfile with new Fabric v2
Support for slack attachments
Improve slack channel handling
skoczen#348 Scheduled reminders should notify the user who requested them
…ckends Add --backends option to generate_will_project
Improve Slack Connection handling
`What time is it` doesn't require question mark. Changed regex so it doesn't require question mark at end of `what time is it in <place>`
| @@ -56,7 +56,7 @@ def get_timezone(lat, lng): | |||
|
|
|||
| class TimePlugin(WillPlugin): | |||
|
|
|||
| @respond_to("what time is it in (?P<place>.*)?\?+") | |||
| @respond_to("what time is it in (?P<place>.*)?(\?)$") | |||
There was a problem hiding this comment.
Couldn't you just do:
@respond_to("what time is it in (?P<place>.*)?\?+?")| @@ -56,7 +56,7 @@ def get_timezone(lat, lng): | |||
|
|
|||
| class TimePlugin(WillPlugin): | |||
|
|
|||
| @respond_to("what time is it in (?P<place>.*)?\?+") | |||
| @respond_to("(what time is it in |time in)(?P<place>.*[^\?])") | |||
There was a problem hiding this comment.
Looking good. Does this first group need to be a capture group?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What time is itdoesn't require question mark.Changed regex so it doesn't require question mark at end of
what time is it in <place>