Skip to content

Commit

Permalink
fix lint
Browse files Browse the repository at this point in the history
Signed-off-by: Raphaël Courivaud <[email protected]>
  • Loading branch information
rcourivaud committed Jun 11, 2024
1 parent de050d7 commit 1461c77
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 4 deletions.
2 changes: 1 addition & 1 deletion analytics/dagster/src/__init__.py
Original file line number Diff line number Diff line change
@@ -1 +1 @@
from .definitions import defs as defs
from .definitions import defs as defs
6 changes: 4 additions & 2 deletions analytics/dagster/src/assets/hackernews.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,9 @@ def topstories(context: AssetExecutionContext) -> MaterializeResult:

results = []
for item_id in topstory_ids:
item = requests.get(f"https://hacker-news.firebaseio.com/v0/item/{item_id}.json").json()
item = requests.get(
f"https://hacker-news.firebaseio.com/v0/item/{item_id}.json"
).json()
results.append(item)

if len(results) % 20 == 0:
Expand Down Expand Up @@ -93,4 +95,4 @@ def most_frequent_words(context: AssetExecutionContext) -> MaterializeResult:
json.dump(top_words, f)

# Attach the Markdown content as metadata to the asset
return MaterializeResult(metadata={"plot": MetadataValue.md(md_content)})
return MaterializeResult(metadata={"plot": MetadataValue.md(md_content)})
2 changes: 1 addition & 1 deletion analytics/dagster/src/definitions.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,4 +13,4 @@

defs = Definitions(
assets=load_assets_from_package_module(assets), schedules=[daily_refresh_schedule]
)
)

0 comments on commit 1461c77

Please sign in to comment.