Skip to content

Let Dependabot open version updates for the workflow and Composer - #46

Merged
cigamit merged 1 commit into
ctrliq:mainfrom
blaipr:add_dependabot_version_updates
Aug 23, 2026
Merged

Let Dependabot open version updates for the workflow and Composer#46
cigamit merged 1 commit into
ctrliq:mainfrom
blaipr:add_dependabot_version_updates

Conversation

@blaipr

@blaipr blaipr commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Problem

There is no .github/dependabot.yml, so no version updates ever open here. This repository has never had a Dependabot pull request of any kind, and the dependency work has been by hand: #42 Fix multiple CVEs and #43 Fix Composer CVE: CVE-2026-59948 both landed in the last weeks, each one a bump someone had to notice first.

The two actions in release.yml are still pinned at v2, four majors behind the v6 the other Ascender repositories run.

What this covers

GitHub Actions. Both uses: in release.yml, pinned to a SHA with the version in a comment, which Dependabot keeps in step.

Composer, /src. composer.json and composer.lock sit together in src/. Twig, PHPMailer, sebastian/diff and composer/composer come from there, and composer/composer is what #43 was.

Weekly on Monday, matching the other repositories. No groups: key, so one pull request per dependency. Dependabot creates the two labels the first time it opens one.

What this leaves out, and why

Docker. Twice over. The three container files are named db-dockerfile, parser-dockerfile and web-dockerfile, which the docker ecosystem does not match, and all three build from rockylinux/rockylinux:9, a floating major with nothing to bump until that major changes. Renaming them to Dockerfile in their own directories would make them visible, but that is a separate change and it would still only offer the major jump.

npm. The only package.json files in the tree are the two inside the vendored copy of highlight.js under src/js/highlight, 513 files that are checked in rather than installed. Pointing npm at them would produce pull requests that cannot be merged as written.

Tests

None: the file is configuration GitHub reads. It parses as YAML and follows the same shape as the config in ascender, which is running.

Covers the two action references in the release workflow, both still pinned
at v2, and the Composer dependencies under src/, where Twig, PHPMailer and
composer itself come from.

Docker and npm are left out, for reasons the file records.
@ciq-it-service-account

ciq-it-service-account commented Aug 23, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@cigamit cigamit self-assigned this Aug 23, 2026
@cigamit cigamit added the enhancement New feature or request label Aug 23, 2026
@cigamit
cigamit merged commit ca88324 into ctrliq:main Aug 23, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Development

Successfully merging this pull request may close these issues.

3 participants