cli icon indicating copy to clipboard operation
cli copied to clipboard

fix perfsprint linter error

Open KentHsu opened this issue 1 year ago • 8 comments

Description

Removed //nolint: perfsprint directive and fixed all perfsprint linter error

Issue reference

Please reference the issue this PR will close: #1374

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • [X] Code compiles correctly
  • [ ] Created/updated tests
  • [ ] Extended the documentation

KentHsu avatar Jul 24 '24 03:07 KentHsu

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

dapr-bot avatar Sep 13 '24 01:09 dapr-bot

This pull request has been automatically closed because it has not had activity in the last 37 days. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

dapr-bot avatar Sep 20 '24 01:09 dapr-bot

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

dapr-bot avatar Mar 28 '25 21:03 dapr-bot

This pull request has been automatically closed because it has not had activity in the last 37 days. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

dapr-bot avatar Apr 04 '25 21:04 dapr-bot

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

dapr-bot avatar May 04 '25 21:05 dapr-bot

@KentHsu could you please apply the suggested change? It'll fix the lint error. TY!

antontroshin avatar May 08 '25 22:05 antontroshin

Hi @antontroshin , it's updated based on your suggestion. Please let me know if there's other issues blocking this PR. Thank you.

KentHsu avatar May 13 '25 14:05 KentHsu

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

dapr-bot avatar Jun 12 '25 14:06 dapr-bot

@holopin-bot @KentHsu Here's a digital badge as a small token of appreciation.

marcduiker avatar Nov 18 '25 10:11 marcduiker

Congratulations @KentHsu, the maintainer of this repository has issued you a badge! Here it is: https://holopin.io/claim/cmi4f0f7b000ejw04egffqlrg

This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account. Or if you're new to Holopin, you can simply sign up with GitHub, which will do the trick!

holopin-bot[bot] avatar Nov 18 '25 10:11 holopin-bot[bot]