Sourcery refactored main branch #1

sourcery-ai · 2023-09-03T19:03:40Z

Branch main refactored by Sourcery.

If you're happy with these changes, merge this Pull Request using the Squash and merge strategy.

See our documentation here.

Run Sourcery locally

Reduce the feedback loop during development by using the Sourcery editor plugin:

Review changes via command line

To manually merge these changes, make sure you're on the main branch, then run:

git fetch origin sourcery/main
git merge --ff-only FETCH_HEAD
git reset HEAD^

Help us improve this pull request!

sourcery-ai · 2023-09-03T19:03:42Z

clean-data.py

-    # Find the index of "Copy code" line
-    copy_code_index = -1
-    for i, line in enumerate(lines):
-        if line.strip() == 'Copy code':
-            copy_code_index = i
-            break
-
+    copy_code_index = next(
+        (i for i, line in enumerate(lines) if line.strip() == 'Copy code'), -1
+    )


Function clean_pinescript refactored with the following changes:

Use the built-in function next instead of a for-loop (use-next)

Use named expression to simplify assignment and conditional (use-named-expression)

Inline variable that is immediately returned (inline-immediately-returned-variable)

This removes the following comments ( why? ):

# Find the index of "Copy code" line

sourcery-ai · 2023-09-03T19:03:42Z

count.py

 directory_path = "PineScripts"

-file_count = sum([len(files) for _, _, files in os.walk(directory_path)])
+file_count = sum(len(files) for _, _, files in os.walk(directory_path))


Lines 25-25 refactored with the following changes:

Replace unneeded comprehension with generator (comprehension-to-generator)

sourcery-ai · 2023-09-03T19:03:42Z

fetch.py


 while True:
-    url = base_url + f'page-{page}/?script_type=strategies&script_access=open&sort=month_popular&route_range=1'
+    url = f'{base_url}page-{page}/?script_type=strategies&script_access=open&sort=month_popular&route_range=1'


Lines 21-42 refactored with the following changes:

Use f-string instead of string concatenation [×2] (use-fstring-for-concatenation)

sourcery-ai · 2023-09-03T19:03:42Z

fetch_editors-picks.py


 async def download_page(session, page):
-    url = base_url + f'page-{page}/?script_type=strategies&script_access=open&sort=month_popular&route_range=1'
+    url = f'{base_url}page-{page}/?script_type=strategies&script_access=open&sort=month_popular&route_range=1'
