Skip to content
This repository was archived by the owner on Oct 19, 2021. It is now read-only.
This repository was archived by the owner on Oct 19, 2021. It is now read-only.

docs: clarify minimum py version #3

Description

@jo3-l

Summary
String concatenation is mostly being done with + currently, if we're looking to support >= Python 3.6 we can use f-strings instead which are a really nice addition to the language.

For example:
https://github.com/SWC-Python-Club/python-dl/blob/48909716618a5bcb93ccb95ad250561fcf38f280/source/downloader.py#L21
Could be refactored to

log.info(f'Downloading "{video.title}"', self.stdscr)

Not a big change, obviously, but it's a bit nicer than + in any case.

Proposed solution

  • clarify minimum required python version somewhere
  • refactor to using f-strings if we're looking to support >= 3.6 only?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions