Skip to content

Cleanup docs, add type annotations, improved class ref - #419

Merged
zliang-akamai merged 2 commits into
linode:devfrom
zliang-akamai:zhiwei/cleanup-obj
Jun 17, 2024
Merged

Cleanup docs, add type annotations, improved class ref#419
zliang-akamai merged 2 commits into
linode:devfrom
zliang-akamai:zhiwei/cleanup-obj

Conversation

@zliang-akamai

Copy link
Copy Markdown
Member

📝 Description

  1. Cleanup docstring to remove unused parameters.
  2. Add some type annotations.
  3. Improve reference to self's class via type(self) to better support class inheritance.

✔️ How to Test

make testunit
make testint

@zliang-akamai
zliang-akamai requested a review from a team as a code owner June 14, 2024 15:23
@zliang-akamai
zliang-akamai requested review from lgarber-akamai and ykim-akamai and removed request for a team June 14, 2024 15:23

@ykim-akamai ykim-akamai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lgarber-akamai lgarber-akamai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Always good to reduce tech debt, nice work!

@zliang-akamai
zliang-akamai merged commit 3aede44 into linode:dev Jun 17, 2024
@zliang-akamai
zliang-akamai deleted the zhiwei/cleanup-obj branch June 17, 2024 21:12
@zliang-akamai zliang-akamai added the improvement for improvements in existing functionality in the changelog. label Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement for improvements in existing functionality in the changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants