Repository navigation
chore: fix / clean up jsdoc comments #1342
Description
Activity
Documentation of constructor parameters of course needs to go with the constructor. Feel free to fix this. But a general description of the class should stick with the class and not be moved to the constructor.
But when we keep the description to the
constructor, it appears for the main class as well. That's how classes work with jsdoc. So you mean I need to go through all the files and fix it 😅. It's not that I don't want to do it but it's gonna take a lot of grind.But when we keep the description to the constructor, it appears for the main class as well
I'm not sure what you mean by this. In the documentation you linked, the example is
/** Class representing a point. */ class Point { /** * Create a point. * @param {number} x - The x value. * @param {number} y - The y value. */ constructor(x, y) { ... } ... }
So clearly the description - in this case "Class representing a point" - should stick with the class rather than being moved to the constructor.
So you mean I need to go through all the files and fix it
You don't need to fix it, but you may fix it. You could consider automatizing this with some RegEx; use whatever you are most comfortable with.
It's not that I don't want to do it but it's gonna take a lot of grind.
It's fine, you don't have to fix it by any means. There are more interesting problems to solve than to fix this :)
I'm sorry, I messed up a little. I came here as soon as I realized I messed up and saw your reply. I'm currently working on adding more shapes to the Geometry folder (like the square, rectangle, triangle, cube, etc.)
Also, are there proper guidelines to write the code in? For example, use this and that for jsdoc, or use x spaces or x tabsize for indentation, etc.
Also, are there proper guidelines to write the code in? For example, use this and that for jsdoc, or use x spaces or x tabsize for indentation, etc.
See our contribution guidelines. We use StandardJS style, so 2 spaces for indentation. As for JSDoc: Just refer to their docs for best practice.
I just realized, nowhere in the
CONTRIBUTING.mddoes it say that I don't have to use the;to end the sentence, so am I allowed to use it? I've seen all files do not use them, I might have missed checking a few.I just realized, nowhere in the
CONTRIBUTING.mddoes it say that I don't have to use the;to end the sentence, so am I allowed to use it? I've seen all files do not use them, I might have missed checking a few.I'm not sure what you mean by this. If you mean using semicolons to delimit statements rather than relying on automatic semicolon insertion, please avoid this to comply with StandardJS style. Of course you have to keep using them in
for-loops or the like though.If you're referring to usage of
;as english punctuation: That is allowed within comments. If you do use it, note that it does not end, but rather separate sentences; it is effectively a compromise between a comma (,) and a colon (:). See e.g. Collins Dictionary for how to use semicolons in English.By sentence, i meant a code line. Thanks for clarifying
@appgurueu please you assign this issue to me
Reacted by Lars Müller@appgurueu
Please assign this issue to me.- changed the title
[-][FEATURE]: jsdoc not working properly when hovering over paremeters[/-][+]chore: fix / clean up jsdoc comments[/+]on Oct 4, 2023 @appgurueu is this issue still open I want to contribute on cleaning up doc string?
@mohmmadAyesh sure, feel free to work on it. (Please don't add redundant comments, though.)
Hey is this still issue available?
I am a beginner, I would love to work on this and I'd also need some guidelines to start my first contribution.We recently launched a token distribution site. The tokens are already live on-chain and can be interacted with through some DEX platforms, so there's practical utility.
We're encountering occasional loading issues with the claim button — not sure if it's a frontend bug or something else. Has anyone seen similar behavior?
Reacted by Rob SimpsonHey @appgurueu is this issue still open I want to contribute!
Hey @appgurueu is this issue still open I want to contribute!
hey this issue is still open or closed ?!
Motivation
Many people who use libraries not only use help of jsdoc for methods, but also for the parameters. So, moving the jsdoc declaration from before
classto beforeconstructorwould definitely help many beginners using this library. Then, the descriptions would show up for the parameters as well.Examples
No response
Possible workarounds
The possible workaround is that instead of moving the jsdoc completely to before
constructor, the description can stay beforeclassbut the@paramneeds to be moved to beforeconstructorAdditional information
as stated by the documentation for classes