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

[Fix issue #5319] Update core.py #5320

Merged
merged 1 commit into from
Jul 1, 2018
Merged

Conversation

fly-high-bj
Copy link
Contributor

With Python 3.7.0, 'async' and 'await' become reserved words, which cause invalid syntax error when running core.py. Renamed 'async' var to 'async_'

With Python 3.7.0, 'async' and 'await' become reserved words, which cause invalid syntax error when running core.py. Renamed 'async' var to 'async_'
@fly-high-bj
Copy link
Contributor Author

This is to address issue #5319
#5319

@fly-high-bj fly-high-bj changed the title Update core.py [Fix issue #5319] Update core.py Jun 29, 2018
@codecov-io
Copy link

codecov-io commented Jun 29, 2018

Codecov Report

Merging #5320 into master will increase coverage by 0.04%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5320      +/-   ##
==========================================
+ Coverage   61.28%   61.32%   +0.04%     
==========================================
  Files         369      369              
  Lines       23483    23483              
  Branches     2717     2717              
==========================================
+ Hits        14391    14401      +10     
+ Misses       9080     9070      -10     
  Partials       12       12
Impacted Files Coverage Δ
superset/views/core.py 72.96% <100%> (ø) ⬆️
superset/connectors/sqla/models.py 78.13% <0%> (+0.79%) ⬆️
superset/db_engine_specs.py 54.28% <0%> (+0.95%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update df5ce50...70addcc. Read the comment docs.

Copy link
Member

@betodealmeida betodealmeida left a comment

Choose a reason for hiding this comment

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

Nice catch, thanks for fixing this!

@mistercrunch mistercrunch merged commit bfa9fff into apache:master Jul 1, 2018
timifasubaa pushed a commit to airbnb/superset-fork that referenced this pull request Jul 25, 2018
With Python 3.7.0, 'async' and 'await' become reserved words, which cause invalid syntax error when running core.py. Renamed 'async' var to 'async_'
wenchma pushed a commit to wenchma/incubator-superset that referenced this pull request Nov 16, 2018
With Python 3.7.0, 'async' and 'await' become reserved words, which cause invalid syntax error when running core.py. Renamed 'async' var to 'async_'
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.28.0 labels Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.28.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants