Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update of the documentation related to latest changes in logging #24072

Merged
merged 1 commit into from
Aug 8, 2022

Conversation

dmatej
Copy link
Contributor

@dmatej dmatej commented Aug 6, 2022

  • I tried to do my best, but it is not perfect (yet), but I will probably add more after I will forget everything and I will try to follow my own documentation :-D
  • partially replaced links by xrefs and <<>> local refs
    • absolutely great thing is that both maven plugin and Eclipse IDE can validate these references, so you immediately know that you broke something
    • the bad thing is that remote links are NOT validated, so I tried to find everything by fulltext search in Eclipse. In current documentation many links don't work, I created issue Documentation - prefer xref and adoc suffix instead of link and html  #24070 to fix it in another PR.
  • indexes of tables, images and examples were added to the result document again (just in admin guide now)
  • I have found yet several bugs, some are new, some are old, all must be fixed. Now I don't want to write any documentation for some time :-D

Direct link to the processed file on GitHub (wow, even GitHub can make it so nice!)
https://github.com/eclipse-ee4j/glassfish/blob/4bb2dda8fb177cdd5252fa5147d25be3f81323f5/docs/administration-guide/src/main/asciidoc/logging.adoc

@dmatej dmatej added this to the 7.0.0 milestone Aug 6, 2022
@dmatej dmatej requested review from OndroMih and arjantijms August 6, 2022 20:56
@dmatej dmatej self-assigned this Aug 6, 2022
@arjantijms arjantijms merged commit bcf60f8 into eclipse-ee4j:master Aug 8, 2022
@dmatej dmatej deleted the logging-docs branch August 8, 2022 12:59
@dmatej dmatej linked an issue Aug 10, 2022 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Documentation - prefer xref and adoc suffix instead of link and html
2 participants