Skip to content

Commit

Permalink
chore(flake8): fix flake8 issues
Browse files Browse the repository at this point in the history
Jupyter also had an issue with the __all__ statement that was fixed.

Signed-off-by: Patrick Avery <[email protected]>
  • Loading branch information
psavery committed May 31, 2022
1 parent ea45740 commit a66127c
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 6 deletions.
6 changes: 3 additions & 3 deletions .flake8
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,6 @@
ignore =
E501

per-file-ignores =
# These directories will always contain "from ... import *"
trame/*:F401,F403
exclude =
# Exclude the examples for now
examples/*
4 changes: 2 additions & 2 deletions docker/scripts/generate_www.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,10 @@ def run(apps_path, out_path, *modules):
cmd = ["python", "-m", "trame.tools.www", "--output", f"{out_path}", *modules]
subprocess.run(cmd)

# FIXME need to generate index.html => {app_name}.html
with open(apps_path, "r") as rf:
apps_dict = json.load(rf)
apps_dict = json.load(rf) # noqa

# FIXME need to generate index.html => {app_name}.html
# for app_name, config in apps_dict.items():
# name = config['app']
# => cp out_path/index.html out_path/{name}.html
Expand Down
3 changes: 2 additions & 1 deletion trame/app/jupyter.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,8 @@

__all__ = [
"show",
"display_iframe" "run",
"display_iframe",
"run",
]


Expand Down

0 comments on commit a66127c

Please sign in to comment.