René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"fixup_pubsub_v1_keywords.py generates incorrect output in some cases ","articleBody":"I just tried the fixup script and ran into a few issues.\r\n\r\n1. It generates this change even though `PublisherClient.publish` does not take the new style parameters:\r\n```diff\r\n-        publisher_client.publish(topic, bytes(json.dumps(message), 'utf-8'))\r\n+        publisher_client.publish(request = {'topic': topic, 'messages': bytes(json.dumps(message), 'utf-8')})\r\n```\r\n2. It generates this change even though `request` and keyword args are documented to be mutually exclusive:\r\n```diff\r\n-        response = subscriber.pull(subscription_path, max_messages=NUM_MESSAGES, timeout=TIMEOUT)\r\n+        response = subscriber.pull(\r\n+            request = {'subscription': subscription_path, 'max_messages': NUM_MESSAGES}, timeout=TIMEOUT,\r\n+        )\r\n```\r\n3. all of the generated `request` keyword arguments break pep8 (they should not have spaces around `=`)\r\n\r\n\r\nStylistically, I find the keyword arguments better to read than all of the inline dicts for the `request` syntax.  It'd be great if the tool had an option to emit the keyword args version.","author":{"url":"https://github.com/rvandegrift","@type":"Person","name":"rvandegrift"},"datePublished":"2020-09-17T22:59:37.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":7},"url":"https://github.com/199/python-pubsub/issues/199"}