Skip to content

Enhance manual - #19

Merged
ensko merged 7 commits into
ensko:mainfrom
alyst:update-manual
Jul 15, 2025
Merged

Enhance manual#19
ensko merged 7 commits into
ensko:mainfrom
alyst:update-manual

Conversation

@alyst

@alyst alyst commented Jul 12, 2025

Copy link
Copy Markdown
Contributor

It is a subset of #18 that only updates the manual (the new API introduced by #18 is not discussed) and the docstrings:

  • expand introduction clarifying Alexandria's prefixes
  • move examples discussion to the relevant section
  • convert full: true to false to condense the manual and better illustrate the numbering behavior
  • tidy up sections numbering, disable Bibliographies numbering

alyst added 3 commits July 12, 2025 14:54
- expand introduction clarifying Alexandria's prefixes
- move example discussion to the relevant section
- convert `full: true` to `false` to condense the manual and better
  illustrate the numbering behavior
- tidy up sections numbering, disable Bibliographies numbering
- hide the internal implementation details
- condense redundancies
- clarify the role of prefixes and read function
it is not required, as alexandria is the only module in the manual

@ensko ensko left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great! I had some small remarks, but after they are addressed I'll merge this :)

(I'll review the other PR probably tomorrow)

Comment thread docs/manual.typ Outdated
Comment thread docs/manual.typ Outdated
Comment thread docs/manual.typ Outdated
Comment thread src/lib.typ
@alyst

alyst commented Jul 15, 2025

Copy link
Copy Markdown
Contributor Author

@SillyFreak Thank you for the review! I have addressed your comments. I've also further slightly tweaked some docstrings (see 8995490), I hope you are ok with these late changes.

@ensko
ensko merged commit ecd4bcb into ensko:main Jul 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants