'Converting' comments to javadoc comment-style #81

Merged
Starfish merged 2 commits from :create-javadoc-comments into master 2 years ago

TL;DR /* -> /**

Hi 👋

this is a rather small change in terms of SLOC.

I learned about code documentation today with javadoc today and spent the day implementing it (as a upcoming gitlab pages project using GitLab CI/CD -> is not yet public).

javadoc is a java commandline tool shipped with e.g. openjdk.

While doing that I realised that with changing the comment style for multiline comments from:

/*
* Lorem Ipsum ...
*
*/

to

/**
* Lorem Ipsum ...
*
*/

we also change to the javadoc comment-style and the comments get integrated in the generated html pages.

I also removed a typo in Astronomy.java that I discovered while reading the generated test data.

**TL;DR** `/*` -> `/**` Hi 👋 this is a rather small change in terms of [SLOC](https://en.wikipedia.org/wiki/Source_lines_of_code). I learned about code documentation today with [javadoc](https://en.wikipedia.org/wiki/Javadoc) today and spent the day implementing it (as a upcoming gitlab pages project using GitLab CI/CD -> is not yet public). `javadoc` is a java commandline tool shipped with e.g. `openjdk`. While doing that I realised that with changing the comment style for multiline comments from: ``` /* * Lorem Ipsum ... * */ ``` to ``` /** * Lorem Ipsum ... * */ ``` we also change to the [`javadoc`](https://docs.oracle.com/javase/8/docs/technotes/tools/windows/javadoc.html) comment-style and the comments get integrated in the generated html pages. I also removed a typo in [`Astronomy.java`](https://codeberg.org/Starfish/TinyWeatherForecastGermany/src/branch/master/app/src/main/java/org/astronomie/info/Astronomy.java#L477) that I discovered while reading the generated test data.
eUgEntOptIc44 added 2 commits 2 years ago
Poster

The named GitLab project is also capable of mirroring the codeberg repo to GitLab and GitHub.

I'd like to share the details with @Starfish upfront to a possible public release in the next weeks.

The named GitLab project is also capable of mirroring the codeberg repo to GitLab and GitHub. I'd like to share the details with @Starfish upfront to a possible public release in the next weeks.
Poster

for further examples concerning the use of javadoc please see the comments in Astronomy.java

This wikipedia table might also be helpful.

for further examples concerning the use of `javadoc` please see the comments in [`Astronomy.java`](https://codeberg.org/Starfish/TinyWeatherForecastGermany/src/branch/master/app/src/main/java/org/astronomie/info/Astronomy.java) This wikipedia [table](https://en.wikipedia.org/wiki/Javadoc#Table_of_Javadoc_tags) might also be helpful.
Starfish merged commit 1477f8c5c4 into master 2 years ago
Owner

I wasn't aware that license info should be Javadoc too. :-)

I wasn't aware that license info should be Javadoc too. :-)
Poster

@Starfish

I wasn't aware that license info should be Javadoc too. :-)

well fair point :-)
Just thought it might be better to streamline the comment-style. But you're right from the javadoc scope there's no real added value at this time

@Starfish >I wasn't aware that license info should be Javadoc too. :-) well fair point :-) Just thought it might be better to streamline the comment-style. But you're right from the javadoc scope there's no real added value at this time
The pull request has been merged as 1477f8c5c4.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Starfish/TinyWeatherForecastGermany#81
Loading…
There is no content yet.