Please be aware that some listings have been temporarily delisted from our marketplace in order to improve the security of the platform. We apologize for any inconvenience this may caused.
For more information on this matter, please visit #1.
If you are a listing owner and would like to have your listing(s) reinstated, please open a ticket at https://gitlab.eclipse.org/eclipsefdn/it/websites/marketplace.eclipse.org/-/issues.
Thank you for your understanding and cooperation.
AgileReview
Details Group Tabs
Friday, March 24, 2023 - 03:07
Reviews Sign in to post reviews
Really nice plugin!
Submitted by Thomas Arnbjerg on Wed, 2019-09-25 08:52
Provides the ability to post review comments in your favorite environment with all the tools available in Eclipse. Beats the online tools by lengths.
Can also be used to annotate the code with comments without poluting the source code.
Never change the source code with your review annotations!
Submitted by Pavan G on Fri, 2018-09-21 06:15
Alpesh Shah above expressed a very sensible feedback and team failed to recognize it. sad. Guys! your tool is wonderful however, you should never fiddle around with source code, instaed have more info in your review.xml file. something like, file name and line number. Its okay, even things dont match when someone update the source and line numbers change. Perhaps you can offer a configuration setting in which user can choose (i) inline code annotations or (ii) no change in source, all info will be in your xml file with warning message that, it might not sync if someone update the code file and line numbers change.
Looking forward to your comments/reply.
Re: Never change the source code with your review annotations!
Submitted by Jake .. on Sat, 2021-01-16 01:41
Can't you connect the source file in such a way that changes to it changes the xml file and keeps it in sync?
Re: Never change the source code with your review annotations!
Submitted by Malte Brunnlieb on Fri, 2019-02-08 03:55
Hi, an opt-out might be a solution for that.
The main focus of that tools was the stability of review comments against refactorings, which could just be assured by adding marks to the code. I really know your fears about letting any tool change your code. Basically, this design decision has been made in the very beginning as of large software projects needs to get refactoring stable code comments.
Anyhow, I will take your feedback into account. Please don't hesitate to add your feature request in the GitHub issue tracker. Thanks.
Re: Never change the source code with your review annotations!
Submitted by Peter Reuter on Thu, 2018-09-27 04:05
Hi Pavan G, see my reply to Alpesh Shah's comment :)
Very useful tool - marker would be appreaciated
Submitted by Alpesh Shah on Mon, 2016-11-28 01:48
I found this tool fulfill almost all requirement for code review within eclipse. Only one point I would like to improve is, for every review comment, an inline comment is added in code which need to be committed along with code. This sometimes clutter the code and it also means that you need to checkout & checkin code even when you don't make any changes to code but just invalidate review comment. Instead of inline comment, if there is just marker on left bar (like how eclipse shows problem or sonar shows violations) could be perfect solution.
Re: Very useful tool - marker would be appreaciated
Submitted by Peter Reuter on Thu, 2018-09-27 04:04
Thanks for your feedback! Linking code and comments, even if the code got refactored, is one of the core features of AgileReview. The easiest way to ensure this is by adding markers in the code. AgileReview was constructed around this principle. Showing markers like problems or sonarlint violations is not working with comments as these markers of other tools result from static code analysis which can be performed on demand. Have you tried tools like Gerrit, which are based on individual commits? Maybe these suit your needs more.
Btw: AgileReview is open source. Feel free to contribute your suggestions as pull requests via GitHub :)
Re: Very useful tool - marker would be appreaciated
Submitted by Pavan G on Fri, 2018-09-21 06:16
Well said!