Skip to content

fix(config): Add deps-dev as valid scope as Dependabot used this often - #422

Merged
halcwb merged 1 commit into
informedica:masterfrom
7sharp9:allow-dependabot-PRs
Jul 21, 2026
Merged

fix(config): Add deps-dev as valid scope as Dependabot used this often#422
halcwb merged 1 commit into
informedica:masterfrom
7sharp9:allow-dependabot-PRs

Conversation

@7sharp9

@7sharp9 7sharp9 commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

Add deps-dev as valid scope as Dependabot used this often to update dependencies, this is used by both the commit hook and CI checks for commit and PR adherence.

@greptile-apps

greptile-apps Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds deps-dev as a valid commit message scope in the Husky commit-lint script, enabling Dependabot's development-dependency update commits to pass linting without being rejected.

  • The change is a one-line addition of deps-dev to the pipe-delimited scope list in the regex pattern used by both the Husky commit hook and CI checks.
  • No logic, pattern structure, or other scopes are altered.

Confidence Score: 5/5

Safe to merge — the change is a single additive entry in a string list and cannot break existing valid commits.

The only change is inserting deps-dev into the pipe-delimited scope regex. Existing scopes and the pattern structure are untouched, so no currently-valid commit messages can be rejected by this change.

No files require special attention.

Important Files Changed

Filename Overview
.husky/scripts/commit-lint.fsx Adds deps-dev to the allowed commit scopes list; trivial additive change with no logic impact. Note: .fsx files are excluded from detailed review per project rules.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Git commit triggered] --> B[Husky pre-commit hook]
    B --> C[commit-lint.fsx runs]
    C --> D{Message matches regex?}
    D -->|yes| E[Validate type]
    E --> F{Type valid?}
    F -->|yes| G[Validate scope]
    G --> H{Scope valid?}
    H -->|deps-dev now allowed| I[Commit accepted]
    H -->|unknown scope| J[Commit rejected]
    F -->|no| J
    D -->|no| J
Loading
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    A[Git commit triggered] --> B[Husky pre-commit hook]
    B --> C[commit-lint.fsx runs]
    C --> D{Message matches regex?}
    D -->|yes| E[Validate type]
    E --> F{Type valid?}
    F -->|yes| G[Validate scope]
    G --> H{Scope valid?}
    H -->|deps-dev now allowed| I[Commit accepted]
    H -->|unknown scope| J[Commit rejected]
    F -->|no| J
    D -->|no| J
Loading

Reviews (1): Last reviewed commit: "fix(config): Add deps-dev as valid scop..." | Re-trigger Greptile

@halcwb
halcwb merged commit e4a1d0c into informedica:master Jul 21, 2026
12 checks passed
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