Skip to content

Enhance GitHub workflows and project configuration - #50

Merged
ptr727 merged 11 commits into
developfrom
workflow
Apr 16, 2026
Merged

Enhance GitHub workflows and project configuration#50
ptr727 merged 11 commits into
developfrom
workflow

Conversation

@ptr727

@ptr727 ptr727 commented Apr 16, 2026

Copy link
Copy Markdown
Owner

Improve GitHub workflows for code generation and auto-merging, enhance security in merge-bot processes, and centralize project configuration with shared MSBuild properties and NuGet package versions. Update documentation to reflect these changes and ensure accurate quote generation in the CodeGen class.

ptr727 and others added 10 commits April 15, 2026 11:02
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…space settings

Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 16, 2026 02:07

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates developer tooling and CI/CD configuration to improve workflow safety/clarity, Docker build ergonomics, and Dependabot auto-merge behavior, aligning repository automation with the intended release/process policies.

Changes:

  • Adjust Dependabot auto-merge gating to allow major updates for non-NuGet ecosystems while still blocking NuGet semver-major auto-merges.
  • Fix Docker build workflow version argument wiring (BUILD_ASSEMBLY_VERSION) and refine Docker-related guidance/ignores.
  • Remove a couple of VS Code workspace settings related to agent request limits and local actions runner command.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

Show a summary per file
File Description
ProjectTemplate.code-workspace Removes workspace settings not essential to repo operation.
Docker/Dockerfile Updates commented build commands to reference the current Dockerfile path and adds a logging example.
.github/workflows/merge-bot-pull-request.yml Refines Dependabot merge condition and improves explanatory comments.
.github/workflows/build-docker-task.yml Corrects BUILD_ASSEMBLY_VERSION to use AssemblyVersion and clarifies Docker Hub login rationale.
.dockerignore Ignores .artifacts/ directories to keep Docker build context smaller and cleaner.

@ptr727
ptr727 enabled auto-merge (squash) April 16, 2026 02:10
@ptr727
ptr727 merged commit 4da840b into develop Apr 16, 2026
20 checks passed
@ptr727
ptr727 deleted the workflow branch May 11, 2026 14:40
ptr727 added a commit that referenced this pull request Jul 18, 2026
…esolved (#348)

Reconciles the ESPHome-Config registry entry after its conformance
standup (ESPHome-Config#46) and the Dependabot ecosystem fix
(ESPHome-Config#49 / promotion #50) completed - same close-out pattern
as #329/#335.

The second `driftNote` recorded the operational rollout as pending and
Dependabot as `devcontainers`-only. Both are resolved and **verified
from the live repo** before dropping the note:

- esphome-config validation job feeds the required check; `version.json`
+ `publish-release.yml` present; **2.0.12** released 2026-07-18;
`develop` is ground truth.
- `.github/dependabot.yml` declares `github-actions` **dual-target** on
both `develop` and `main`.
- `python3 spec/audit.py ESPHome-Config` (with the #346 check now on
develop) -> **clean**; no `dependabot` DRIFT.

The first note (ESPHome device config consumed by the ESPHome-NonRoot
image) stays - still true. `spec/validate.py` passes.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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