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

Fix param names in docstrings and remove redundant parentheses #6014

Closed
wants to merge 1 commit into from

Conversation

KeyLee123
Copy link

Optimize the codes, fix param names in docstrings and remove redundant parentheses.

@mr-avocado
Copy link

mr-avocado bot commented Aug 29, 2024

Dear contributor,
Avocado is currently at the end of sprint 107, therefore we are in feature freeze state.
Please avoid merging changes that do not fall into these categories:

  • Bug fixes
  • Documentation updates

The feature freeze will be active until the release planned on September 02, 2024.

Copy link
Contributor

@richtja richtja left a comment

Choose a reason for hiding this comment

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

Hi @KeyLee123 welcome to the avocado community, we are happy to have you here. Your changes LGTM, you just need to address few commit style requirements.

  • At the end of your commit please add your signature. In your case it is Signed-off-by: lixu <[email protected]>
  • Your changes are located only to the avocado/utils/cpu.py. Please write this into the commit title to make it more specific.
  • It looks like the oldfuncname is unknown to are spell-check. Please update the spell.ignore file with this term.

@KeyLee123 KeyLee123 closed this Aug 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants