Hello SonarSource team
Here is the related PR:
Welcome to the community!
I want to let you know that I’ve seen this request. It’s on my list, but there are a few other things I need to get out of the way first. I hope to come back to you this week, though.
I’m delighted to be part of the community!
Thank you for this message. There’s no hurry, I can wait.
I look forward to your feedback
The bureaucratic requirements are met.
The testing looks good as well, so I’m going to process your PR. Just a few notes:
And BTW, I’ve already provided feedback internally about the table cell vertical alignment.
It looks like all your rules have been classed as Consistency/Conventional in the new Clean Code taxonomy. I’m (now) assuming that’s the default. Just a note that in future releases you may want to take a look at this.
One thing I’m not seeing is
// Noncompliant comments in your noncompliant code samples. They help users separate the wheat from the chaff
limit and not a
where clause. I would think limiting CPU at the server / DB would be in scope here
I see that a few rules provide Further Reading, which is awesome. And there’s a way to mark that up to have it shown on a ‘More Info’ tab, E.G.. I’m not aware of the details, but if you want to create a thread, we can get it tracked down.
plural noun: assignations
an appointment to meet someone in secret, typically one made by lovers.
You want “assignments”
Thank you for your time and for the quick review!
It’s very pleasing to see your commitment to the plugins made by the community.
Indeed, there are still a few adjustments to be made and this translation is clearly not good haha.
I will create an issue on the project repository to process all your highly relevant feedback!
All the best,
sorry to disturb you again but the update repository https://update.sonarsource.org does not appear to have updated since October 9. I have seen this date at the top of this file.
Do you know if there is a problem somewhere? Or when the site will be updated?
Thanks in advance and have a nice evening!
Thanks for the ping, and sorry about this. I promise the job has been run, but internally, we’ve got someone “helpfully” moving the job to a “better” provider. This is also currently screwing up showing SonarQube 10.3 in the Plugin Version Matrix.
Hopefully we’ll get this sorted out soon.
We’ve tracked down the problem & it should be fixed. Thanks again for your ping on this.
Thanks for the quick fix! Looks good from my end.
Have a nice weekend!
This topic was automatically closed 7 days after the last reply. New replies are no longer allowed.