You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The MD report generated by the extension cannot be easily saved. We should make it possible, as well as have an option to define a default report location or something like this.
The text was updated successfully, but these errors were encountered:
Absolutely on the default location. In fact, why not make it into the root of the project so that it can be checked in and shared with team members. While annotations may be private reminders, they can also serve as code review notes.
@tomotvos That's one interesting idea, which I must say is the opposite of my use case hahaha. In my case, I don't want anything visible that could be easily committed by mistake.
But your case makes a lot of sense, and maybe my use case is the exception. But allowing it to be configurable, with a default value as the root can support both user cases :)
I have just discovered the Code Annotation extension and I am currently exploring it for planning refactoring of a project.
I am going through the code base and making notes on which parts I think should be moved to other locations etc. I find it too messy to litter my source files with comments and check those in. I like the idea much better of being able to keep my comments on this separate from the actual source code, yet clearly linked to specific places in the source code.
Your extension @thamara looks quite fit for this purpose. Like @tomotvos, I would also like to be able to share these notes with my colleagues, so I was looking to see if the extension stores the notes in a sidecar file somewhere in my repo that I could just commit and let colleagues use with the extension on their computers.
The MD report generated by the extension cannot be easily saved. We should make it possible, as well as have an option to define a default report location or something like this.
The text was updated successfully, but these errors were encountered: