Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor modules and update code format. (#49) #1

Merged
merged 1 commit into from
Oct 4, 2022

Conversation

Exoutia
Copy link
Owner

@Exoutia Exoutia commented Oct 4, 2022

  • update tests/ with standardized code format

  • rm extra newline

  • update f1pystats/ with standardized code format and more concise function calls

  • fix bug - add f1pystats.finishing_status

  • fix lines exceeding 100 char

  • Removed unnecessary comma

  • apply timeout kwarg when requesting JSON content

  • Fixed timeout issue

  • Changed order of some functions

Placed _get_json_content_from_url at the bottom of the f1pystats.py file.

Co-authored-by: alec-kr [email protected]

Description

Related Issue(s)

Fixes #X

User-facing Changes

Screenshots (If necessary)

* update tests/ with standardized code format

* rm extra newline

* update f1pystats/ with standardized code format and more concise function calls

* fix bug - add f1pystats.finishing_status

* fix lines exceeding 100 char

* Removed unnecessary comma

* apply timeout kwarg when requesting JSON content

* Fixed timeout issue

* Changed order of some functions

Placed ```_get_json_content_from_url``` at the bottom of the ```f1pystats.py``` file.

Co-authored-by: alec-kr <[email protected]>
@Exoutia Exoutia merged commit 0d7ac27 into Exoutia:adding-get_drivers() Oct 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants