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

Add new basic tests TestDocsGenerateDatabricks and TestDocsGenReferencesDatabricks #134

Merged
merged 4 commits into from
Jul 21, 2022

Conversation

ueshin
Copy link
Collaborator

@ueshin ueshin commented Jul 18, 2022

Description

Adds new basic tests TestDocsGenerateDatabricks and TestDocsGenReferencesDatabricks based on BaseDocsGenerate and BaseDocsGenReferences respectively as per an item from #127.

@@ -5,7 +5,7 @@ envlist = linter, unit
[testenv:linter]
basepython = python3
commands = /bin/bash -c '$(which black) --config black.ini --check .'
/bin/bash -c '$(which flake8) --select=E,W,F --ignore=E203 --max-line-length=100 .'
/bin/bash -c '$(which flake8) --select=E,W,F --ignore=E203,W503 --max-line-length=100 .'
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

To fix the compatibility issue with black formatter:

./tests/functional/adapter/test_basic.py:99:13: W503 line break before binary operator
./tests/functional/adapter/test_basic.py:100:13: W503 line break before binary operator
./tests/functional/adapter/test_basic.py:102:17: W503 line break before binary operator

@ueshin
Copy link
Collaborator Author

ueshin commented Jul 21, 2022

Thanks! merging.

@ueshin ueshin merged commit 7e15f1e into databricks:main Jul 21, 2022
@ueshin ueshin deleted the doc_gen branch July 21, 2022 18:25
ueshin added a commit that referenced this pull request Jul 21, 2022
…cesDatabricks (#134)

### Description

Adds new basic tests `TestDocsGenerateDatabricks` and `TestDocsGenReferencesDatabricks` based on `BaseDocsGenerate` and `BaseDocsGenReferences` respectively as per an item from #127.
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