Skip to content

Document Docker image and action usage#3544

Merged
koxudaxi merged 1 commit into
mainfrom
docs/docker-action-followups
Jul 2, 2026
Merged

Document Docker image and action usage#3544
koxudaxi merged 1 commit into
mainfrom
docs/docker-action-followups

Conversation

@koxudaxi

@koxudaxi koxudaxi commented Jul 2, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Documentation
    • Updated Docker usage guidance to note that published images run as a non-root user and to help ensure bind-mounted output directories are writable.
    • Refreshed CI/CD examples across the docs to use the latest recommended GitHub Action version.
    • Updated version-pinning tips and workflow snippets to match the newer release.

@coderabbitai

coderabbitai Bot commented Jul 2, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 6ea297b7-6e86-4168-a277-d8f8ba185460

📥 Commits

Reviewing files that changed from the base of the PR and between a437c7d and d39a794.

⛔ Files ignored due to path filters (1)
  • docs/llms-full.txt is excluded by none and included by none
📒 Files selected for processing (3)
  • README.md
  • docs/ci-cd.md
  • docs/index.md

📝 Walkthrough

Walkthrough

Documentation updates pin the koxudaxi/datamodel-code-generator GitHub Action reference from version 0.44.0 to 0.66.3 across README.md, docs/ci-cd.md, and docs/index.md workflow examples. README.md additionally adds guidance about the Docker image running as a non-root appuser and writable bind-mount directories.

Changes

Documentation updates

Layer / File(s) Summary
Docker non-root user guidance
README.md
Adds a note explaining the container runs as non-root appuser and advises ensuring writable bind-mounted output directories or passing an explicit --user.
GitHub Action version pin updates
README.md, docs/ci-cd.md, docs/index.md
Updates all pinned koxudaxi/datamodel-code-generator action version references from @0.44.0 to @0.66.3 across README and CI/CD documentation workflow examples.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Poem
A rabbit hopped through docs today,
Bumped version tags from old to new,
Non-root appuser, safe and true,
Bind mounts writable, permissions fine,
Hop, hop, hooray — the docs align!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main documentation updates for Docker image guidance and GitHub Action usage examples.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/docker-action-followups

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

📚 Docs Preview: https://pr-3544.datamodel-code-generator.pages.dev

@codspeed-hq

codspeed-hq Bot commented Jul 2, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

✅ 11 untouched benchmarks
⏩ 98 skipped benchmarks1


Comparing docs/docker-action-followups (d39a794) with main (a437c7d)

Open in CodSpeed

Footnotes

  1. 98 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@codecov

codecov Bot commented Jul 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (a437c7d) to head (d39a794).

Additional details and impacted files
@@            Coverage Diff            @@
##              main     #3544   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          161       161           
  Lines        34363     34363           
  Branches      4011      4011           
=========================================
  Hits         34363     34363           
Flag Coverage Δ
unittests 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@koxudaxi koxudaxi marked this pull request as ready for review July 2, 2026 16:24
@koxudaxi koxudaxi merged commit 33476e6 into main Jul 2, 2026
61 checks passed
@koxudaxi koxudaxi deleted the docs/docker-action-followups branch July 2, 2026 16:28
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Breaking Change Analysis

Result: No breaking changes detected

Reasoning: PR #3544 is documentation-only. It modifies README.md, docs/ci-cd.md, docs/index.md, and docs/llms-full.txt to (1) add a note that published Docker images run as a non-root appuser and (2) bump example GitHub Action version pins in docs from @0.44.0 to @0.66.3. There are no changes to generated code output, Jinja2 templates, CLI options, the Python API, default values/behavior, supported Python versions, or error handling. The deleted lines are only the old doc version strings. None of the breaking-change categories apply.


This analysis was performed by Claude Code Action

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

🎉 Released in 0.67.0

This PR is now available in the latest release. See the release notes for details.

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.

1 participant