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

Restyle Example Script: Get table names used by data source #22

Closed
wants to merge 26 commits into from

Conversation

restyled-io[bot]
Copy link

@restyled-io restyled-io bot commented Apr 17, 2020

Automated style fixes for #11, created by Restyled.

The following restylers made fixes:

  • autopep8
  • black
  • yapf

To incorporate these changes, merge this Pull Request into the original. We
recommend using the Squash or Rebase strategies.

NOTE: As work continues on the original Pull Request, this process will
re-run and update (force-push) this Pull Request with updated style fixes as
necessary. If the style is fixed manually at any point (i.e. this process finds
no fixes to make), this Pull Request will be closed automatically.

Sorry if this was unexpected. To disable it, see our documentation.

susodapop and others added 23 commits April 13, 2020 13:33
HT: @rauchy

Co-Authored-By: Omer Lachish <[email protected]>
and table name. Sets the re.UNICODE flag as suggested by @kravets-levko
It's not a problem to return a qualified table name.
* Restyled by autopep8
* Restyled by black
* Restyled by reorder-python-imports
* Restyled by yapf

Co-authored-by: Restyled.io <[email protected]>
@susodapop
Copy link
Collaborator

This restyle disagrees with Black.

Copy link
Collaborator

@susodapop susodapop left a comment

Choose a reason for hiding this comment

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

I'm planning to stick with just Black for the finished version.

@click.argument(
"url", )
@click.argument(
"key", )
@click.argument("data_source_id")
Copy link
Collaborator

Choose a reason for hiding this comment

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

This seems less readable to me.

in the filename. Otherwise the example is impossible to import.
Base automatically changed from issue-10 to master February 24, 2021 17:44
@susodapop susodapop closed this Feb 25, 2021
@susodapop susodapop deleted the restyled/issue-10 branch February 25, 2021 19:38
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.

2 participants