nox icon indicating copy to clipboard operation
nox copied to clipboard

fix: typos in error message, metrics descriptions and comments

Open cuinix opened this issue 1 year ago • 4 comments

Description

fix some comments

Motivation

[Explain why these changes are necessary or beneficial. Discuss any problems or limitations with the current implementation that this pull request addresses.]

Related Issue(s)

[Cite any related issues or feature requests here, using GitHub issue links.]

Proposed Changes

[List the specific changes and additions introduced by this pull request.]

Screenshots (if applicable)

[Add any relevant screenshots or animated GIFs to showcase the changes.]

Additional Notes

[Provide any additional information or context that may be helpful for the reviewer.]

Checklist

  • [x] The code follows the project's coding conventions and style guidelines.
  • [x] All tests related to the changes have passed successfully.
  • [x] Documentation has been updated to reflect the changes (if applicable).
  • [x] All new and existing unit tests have passed.
  • [x] I have self-reviewed my code and ensured its quality.
  • [x] I have added/updated necessary comments to aid understanding.

Reviewer Checklist

  • [ ] Code has been reviewed for quality and adherence to guidelines.
  • [ ] Tests have been reviewed and are sufficient to validate the changes.
  • [ ] Documentation has been reviewed and is up to date.
  • [ ] Any questions or concerns have been addressed.

cuinix avatar Mar 06 '24 07:03 cuinix

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Mar 06 '24 07:03 CLAassistant

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Mar 06 '24 07:03 CLAassistant

Anything need i do?

cuinix avatar Apr 20 '24 11:04 cuinix

Hello, @cuinix! Sorry for the long response.

Could you kindly resolve conflicts, so we run the tests and accept the PR? Thank you!

folex avatar May 21 '24 09:05 folex