a few small improvements to the javadoc source code documentation #138

Merged
Starfish merged 2 commits from eUgEntOptIc44/TinyWeatherForecastGermany:javadoc-package-info into master 8 months ago

Hi @Starfish,

As the title of this PR reads I recently learned about the role of package-info.java within javadoc. It introduces the possibility of adding a short1 description to the package overview on the index page of the javadoc.

It's only a little change but it'd fill the blank space right in the middle of the page.

Screenshot of javadoc index page

I also found a few typos in the source code and fixed them.

Looking forward to your feedback on this.

Regards,

Jean-Luc


  1. only one line ↩︎

Hi @Starfish, As the title of this PR reads I recently learned about the role of `package-info.java` within javadoc. It introduces the possibility of adding a short[^1] description to the package overview on the index page of the javadoc. It's only a little change but it'd fill the blank space right in the middle of the page. ![Screenshot of javadoc index page](https://codeberg.org/attachments/df2013e1-b5d3-4400-8e84-b59a7e1259f1) I also found a few typos in the source code and fixed them. Looking forward to your feedback on this. Regards, Jean-Luc [^1]: only one line
eUgEntOptIc44 added 2 commits 8 months ago
Starfish merged commit 0e116f73b8 into master 8 months ago
The pull request has been merged as 0e116f73b8.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b eUgEntOptIc44-javadoc-package-info master
git pull javadoc-package-info

Step 2:

Merge the changes and update on Forgejo.
git checkout master
git merge --no-ff eUgEntOptIc44-javadoc-package-info
git push origin master
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
1 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#138
Loading…
There is no content yet.