Skip to content

gvegayon's patch to Dev specify versions description mpw#363

Closed
gvegayon wants to merge 23 commits intomainfrom
dev-specify-versions-description-mpw-ggvy
Closed

gvegayon's patch to Dev specify versions description mpw#363
gvegayon wants to merge 23 commits intomainfrom
dev-specify-versions-description-mpw-ggvy

Conversation

@gvegayon
Copy link
Copy Markdown
Member

@gvegayon gvegayon commented Aug 27, 2025

This pull request introduces a new development container setup for the project, making it easier to build, test, and develop in a consistent environment. The changes include a new Dockerfile for the dev container, supporting scripts, configuration files, and updates to CI workflows to leverage the container for building and deploying documentation. Additionally, minor improvements and fixes have been made to package metadata and the Makefile.

Development Environment Setup:

  • Added .devcontainer/Containerfile to define a reproducible Docker-based development environment, including installation of R dependencies, building the package, and optional development tools and scripts.
  • Added .devcontainer/devcontainer.json to configure VS Code integration, extensions, and workspace mounts for the dev container.
  • Added .devcontainer/extra.sh to automate installation of additional development dependencies and tools when in development mode.

Continuous Integration / Deployment:

  • Updated .github/workflows/pkgdown.yaml to build the pkgdown website using the new container image, including dynamic image tagging and improved setup steps. [1] [2]
  • Modified .github/workflows/containers-and-az-pool.yaml to add a pkgdown job that uses the new workflow.

Project Metadata and Build System:

  • Fixed the R version requirement in DESCRIPTION to use the correct format.
  • Improved Makefile portability by making the container manager (CNTR_MGR) configurable via environment variable.
  • Updated NEWS.md to document the addition of package version specification in DESCRIPTION.

@gvegayon gvegayon changed the base branch from dev-specify-versions-description-mpw to main August 27, 2025 18:49
@gvegayon
Copy link
Copy Markdown
Member Author

Hello friends, this has a few conflicts, and I am unsure if you will ever want to merge this in, @micahwiesner67. I'm OK if not.

@gvegayon gvegayon marked this pull request as ready for review November 19, 2025 19:46
@gvegayon
Copy link
Copy Markdown
Member Author

gvegayon commented Dec 3, 2025

OK, closing on my side; feel free to open @micahwiesner67

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.

3 participants