Sourcery Starbot ⭐ refactored ryneandal/pokeapi by SourceryAI · Pull Request #1 · ryneandal/pokeapi

SourceryAI · 2022-08-02T19:36:05Z

Thanks for starring sourcery-ai/sourcery ✨ 🌟 ✨

Here's your pull request refactoring your most popular Python repo.

If you want Sourcery to refactor all your Python repos and incoming pull requests install our bot.

Review changes via command line

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

git fetch https://github.com/sourcery-ai-bot/pokeapi master
git merge --ff-only FETCH_HEAD
git reset HEAD^

SourceryAI

Due to GitHub API limits, only the first 60 comments can be shown.

SourceryAI · 2022-08-02T19:36:06Z

 from multiprocessing import cpu_count

-bind = "0.0.0.0:{}".format(os.environ.get("SERVER_PORT", "80"))
+bind = f'0.0.0.0:{os.environ.get("SERVER_PORT", "80")}'


Lines 4-4 refactored with the following changes:

Replace call to format with f-string (use-fstring-for-formatting)

SourceryAI · 2022-08-02T19:36:06Z


 MEDIA_DIR = "/media/sprites/{0}"
-IMAGE_DIR = os.getcwd() + "/data/v2/sprites/sprites/"
+IMAGE_DIR = f"{os.getcwd()}/data/v2/sprites/sprites/"


Lines 34-34 refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

SourceryAI · 2022-08-02T19:36:06Z

    with context:
-        for value in iterable:
-            yield value
+        yield from iterable


Function with_iter refactored with the following changes:

Replace yield inside for loop with yield from (yield-from)

SourceryAI · 2022-08-02T19:36:06Z

-    print("building " + table_name)
+    print(f"building {table_name}")


Function clear_table refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

SourceryAI · 2022-08-02T19:36:07Z

-            if len(sub) >= 2:
-                sub = sub[1]
-            else:
-                sub = sub[0]
+            sub = sub[1] if len(sub) >= 2 else sub[0]


Function scrub_str refactored with the following changes:

Replace if statement with if expression (assign-if-exp)

SourceryAI · 2022-08-02T19:36:08Z

-                past_obj = {}
+                past_obj = {"generation": poke_past_type["generation"]}

-                # create past types object for this generation
-                past_obj["generation"] = poke_past_type["generation"]


Function PokemonDetailSerializer.get_past_pokemon_types refactored with the following changes:

Merge dictionary assignment with declaration (merge-dict-assign)

This removes the following comments ( why? ):

# create past types object for this generation

SourceryAI · 2022-08-02T19:36:08Z

-        groups = []
-        for group in data:
-            groups.append(group["egg_group"])
-
-        return groups
+        return [group["egg_group"] for group in data]


Function PokemonSpeciesDetailSerializer.get_pokemon_egg_groups refactored with the following changes:

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

Convert for loop into list comprehension (list-comprehension)

SourceryAI · 2022-08-02T19:36:08Z

-        # serialize chain recursively from tree
-        chain = self.build_chain_link_entry(evolution_tree, summary_data)
-
-        return chain
+        return self.build_chain_link_entry(evolution_tree, summary_data)


Function EvolutionChainDetailSerializer.build_chain refactored with the following changes:

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

This removes the following comments ( why? ):

# serialize chain recursively from tree

SourceryAI · 2022-08-02T19:36:09Z

-            while len(search_stack) > 0:
+            while search_stack:


Function EvolutionChainDetailSerializer.build_evolution_tree refactored with the following changes:

Simplify sequence length comparison (simplify-len-comparison)

SourceryAI · 2022-08-02T19:36:09Z

-        results = []
-
-        for dex_group in dex_groups:
-            results.append(dex_group["version_group"])
-
-        return results
+        return [dex_group["version_group"] for dex_group in dex_groups]


Function PokedexDetailSerializer.get_pokedex_version_groups refactored with the following changes:

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

Convert for loop into list comprehension (list-comprehension)

SourceryAI · 2022-08-02T19:36:13Z

-        regions = []
-
-        for region in data:
-            regions.append(region["region"])
-
-        return regions
+        return [region["region"] for region in data]


Function VersionGroupDetailSerializer.get_version_group_regions refactored with the following changes:

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

Convert for loop into list comprehension (list-comprehension)

SourceryAI · 2022-08-02T19:36:13Z

-        methods = []
-
-        for method in learn_method_data:
-            methods.append(method["move_learn_method"])
-
-        return methods
+        return [method["move_learn_method"] for method in learn_method_data]


Function VersionGroupDetailSerializer.get_learn_methods refactored with the following changes:

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

Convert for loop into list comprehension (list-comprehension)

SourceryAI · 2022-08-02T19:36:13Z

-        results = []
-
-        for dex_group in dex_groups:
-            results.append(dex_group["pokedex"])
-
-        return results
+        return [dex_group["pokedex"] for dex_group in dex_groups]


Function VersionGroupDetailSerializer.get_version_groups_pokedexes refactored with the following changes:

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

Convert for loop into list comprehension (list-comprehension)

SourceryAI · 2022-08-02T19:36:13Z

    def setup_language_name_data(cls, language, name="lang nm"):

-        local_language = cls.setup_language_data(name="lang for " + name)
+        local_language = cls.setup_language_data(name=f"lang for {name}")


Function APIData.setup_language_name_data refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

SourceryAI · 2022-08-02T19:36:14Z

    def setup_region_name_data(cls, region, name="reg nm"):

-        language = cls.setup_language_data(name="lang for " + name)
+        language = cls.setup_language_data(name=f"lang for {name}")


Function APIData.setup_region_name_data refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

SourceryAI · 2022-08-02T19:36:15Z

    ):

-        language = cls.setup_language_data(name="lang for " + description)
+        language = cls.setup_language_data(name=f"lang for {description}")


Function APIData.setup_item_attribute_description_data refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

SourceryAI · 2022-08-02T19:36:15Z

    ):

-        language = cls.setup_language_data(name="lang for " + effect)
+        language = cls.setup_language_data(name=f"lang for {effect}")


Function APIData.setup_item_fling_effect_effect_text_data refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

SourceryAI · 2022-08-02T19:36:15Z

    def setup_item_pocket_name_data(cls, item_pocket, name="itm pkt nm"):

-        language = cls.setup_language_data(name="lang for " + name)
+        language = cls.setup_language_data(name=f"lang for {name}")


Function APIData.setup_item_pocket_name_data refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

SourceryAI · 2022-08-02T19:36:15Z

        item_pocket = item_pocket or cls.setup_item_pocket_data(
-            name="itm pkt for " + name
+            name=f"itm pkt for {name}"
        )



Function APIData.setup_item_category_data refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

SourceryAI · 2022-08-02T19:36:15Z

-        language = cls.setup_language_data(name="lang for " + name)
+        language = cls.setup_language_data(name=f"lang for {name}")


Function APIData.setup_item_category_name_data refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

SourceryAI · 2022-08-02T19:36:19Z

-        language = cls.setup_language_data(name="lang for " + name)
+        language = cls.setup_language_data(name=f"lang for {name}")


Function APIData.setup_item_name_data refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

SourceryAI · 2022-08-02T19:36:19Z

-        language = cls.setup_language_data(name="lang for " + effect)
+        language = cls.setup_language_data(name=f"lang for {effect}")


Function APIData.setup_item_effect_text_data refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

SourceryAI · 2022-08-02T19:36:19Z

-        version_group = cls.setup_version_group_data(name="ver grp for " + flavor_text)
+        version_group = cls.setup_version_group_data(name=f"ver grp for {flavor_text}")

-        language = cls.setup_language_data(name="lang for " + flavor_text)
+        language = cls.setup_language_data(name=f"lang for {flavor_text}")


Function APIData.setup_item_flavor_text_data refactored with the following changes:

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

SourceryAI · 2022-08-02T19:36:19Z

-        language = cls.setup_language_data(name="lang for " + name)
+        language = cls.setup_language_data(name=f"lang for {name}")


Function APIData.setup_contest_type_name_data refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

SourceryAI · 2022-08-02T19:36:20Z

-        language = cls.setup_language_data(name="lang for " + flavor_text)
+        language = cls.setup_language_data(name=f"lang for {flavor_text}")


Function APIData.setup_contest_effect_flavor_text_data refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

SourceryAI · 2022-08-02T19:36:21Z

-        language = cls.setup_language_data(name="lang for " + name)
+        language = cls.setup_language_data(name=f"lang for {name}")


Function APIData.setup_location_area_name_data refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

SourceryAI · 2022-08-02T19:36:21Z

-            name="mv dmg cls for " + name
+            name=f"mv dmg cls for {name}"
        )

-        generation = generation or cls.setup_generation_data(name="rgn for " + name)
+
+        generation = generation or cls.setup_generation_data(name=f"rgn for {name}")


Function APIData.setup_type_data refactored with the following changes:

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

SourceryAI · 2022-08-02T19:36:21Z

-        language = cls.setup_language_data(name="lang for " + name)
+        language = cls.setup_language_data(name=f"lang for {name}")


Function APIData.setup_type_name_data refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

SourceryAI · 2022-08-02T19:36:22Z

-        language = cls.setup_language_data(name="lang for " + name)
+        language = cls.setup_language_data(name=f"lang for {name}")


Function APIData.setup_move_ailment_name_data refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

SourceryAI · 2022-08-02T19:36:22Z

-        language = cls.setup_language_data(name="lang for " + name)
+        language = cls.setup_language_data(name=f"lang for {name}")


Function APIData.setup_move_battle_style_name_data refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

'Refactored by Sourcery'

f8ea67e

SourceryAI commented Aug 2, 2022

View reviewed changes

		print("building " + table_name)
		print(f"building {table_name}")

		language = cls.setup_language_data(name="lang for " + name)
		language = cls.setup_language_data(name=f"lang for {name}")

Conversation

SourceryAI commented Aug 2, 2022

Uh oh!

SourceryAI left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant