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

Implement the vast majority of QRZ's APIs & general linting/PEP8 compliance #2

Merged
merged 34 commits into from
Nov 24, 2024

Conversation

JayToTheAy
Copy link
Owner

@JayToTheAy JayToTheAy commented Nov 17, 2024

Also bumping this up to a 1.0.0 alpha release

TO-DO:

  • Implement paged fetching for QRZ logbooks
  • Implement inserting QRZ logbook records
  • Implement linting for the repo
  • Make the repo linting compliant... (enough to pass, anyway)
  • Raise for all requests.raise_for_status(), just to help with docstrings and linting detecting exception locations
  • More specific exceptions, at least per-module

What this ISN'T:

  • Any other modules becoming feature-complete
  • Refactors to satisfy linting warnings where it isn't part of QRZ.py
  • Any pyink compliance changes

Resolves #4

@JayToTheAy JayToTheAy changed the title Implement the vast majority of QRZ's APIs Implement the vast majority of QRZ's APIs & general linting/PEP8 compliance Nov 17, 2024
@JayToTheAy JayToTheAy marked this pull request as draft November 17, 2024 05:02
@JayToTheAy
Copy link
Owner Author

this is getting squeamishly big. I might just push this with linting being for a second PR.

@JayToTheAy
Copy link
Owner Author

More QRZ API work will be in another PR for 1.0.0 final.

Resolves #4

@JayToTheAy JayToTheAy added this to the 1.0.0 milestone Nov 24, 2024
@JayToTheAy JayToTheAy self-assigned this Nov 24, 2024
@JayToTheAy JayToTheAy marked this pull request as ready for review November 24, 2024 00:41
@JayToTheAy JayToTheAy merged commit f01a105 into main Nov 24, 2024
9 checks passed
@JayToTheAy JayToTheAy deleted the qrz-api branch November 24, 2024 00:42
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.

linting compliance
1 participant