Skip to content

Team manual back2tex #2242

New issue

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

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

Already on GitHub? # to your account

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from
Draft

Team manual back2tex #2242

wants to merge 9 commits into from

Conversation

thijskh
Copy link
Member

@thijskh thijskh commented Nov 24, 2023

No description provided.

@eldering
Copy link
Member

Some general comments that not necessarily should be done in this PR:

  • It would be nice if doc/team/gentexconfig.in would fetch some configuration defaults from etc/db-config.yaml or if the system is already installed from a running DOMjudge API.
  • I'd make the only make targets under doc/ to be dist, to pre-generate documentation to be shipped in the tarball, no targets that are meant to be run by the admin user installing DOMjudge. I think this makes things clearer and also simplifies the Makefiles.
  • For the team manual, we should just provide a script (basically doc/team/genteammanual) that an admin user can run to regenerate the team manual with updated text and/or configuration. But this should be completely outside the Makefile structure.

@vmcj
Copy link
Member

vmcj commented Nov 21, 2024

@imylene13 is willing to look into this, do we have a specifc reason why we also include the teammanual in the admin manual?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants