Skip to content

build: add Dockerfile for local build environment #532

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

skytomo221
Copy link

Added Docker Support for Building the Reference Manual

This pull request introduces the following changes:

  • Added a Dockerfile: Simplifies the setup process for building the reference manual locally.
  • Updated the README: Includes instructions for building and running the manual using Docker.

Motivation

These changes aim to make it easier for contributors and users to build the reference manual without needing to install dependencies manually. Docker provides a consistent and isolated environment for building the project.

Changes

  1. Dockerfile:
    • Configured to build the reference manual in a containerized environment.
    • Ensures all necessary dependencies are installed.
  2. README Updates:
    • Added a section explaining how to use Docker to build and run the manual.

Related Issue

This pull request is not associated with any existing issue.

- Added a Dockerfile to simplify the setup process for building the reference manual locally.
- Updated the README with instructions for building and running the manual using Docker.
@david-christiansen
Copy link
Collaborator

I'm a bit nervous about adding a new build method that's not part of CI and that I myself don't regularly use. Users may come to depend on it, which is not great if it bit-rots.

What's the main benefit of Docker here? If it's just making the TeX parts easier, then I plan to just check in the figures along with their source. Does this make it easier to build it and check the examples on Windows?

@david-christiansen
Copy link
Collaborator

(and sorry for the slow response, this PR arrived just as I left on summer vacation)

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