Bump senzing-factory/build-resources/.github/workflows/lint-workflows.yaml from 3 to 4

[dependabot]

Bumps senzing-factory/build-resources/.github/workflows/lint-workflows.yaml from 3 to 4.

Release notes

Sourced from senzing-factory/build-resources/.github/workflows/lint-workflows.yaml's releases.

4.0.0

What's Changed

• #260 make shared workflows generic, misc cleanup by @​kernelsam in senzing-factory/build-resources#261

Full Changelog: senzing-factory/build-resources@v3...4.0.0

3.0.31

What's Changed

• Bump super-linter/super-linter from 8.4.0 to 8.5.0 by @​dependabot[bot] in senzing-factory/build-resources#257
• if the PR's REFRESHED_AT matches main's value, it only passes if that… by @​kernelsam in senzing-factory/build-resources#259

Full Changelog: senzing-factory/build-resources@v3...3.0.31

3.0.30

What's Changed

• Bump super-linter/super-linter from 8.3.1 to 8.3.2 by @​dependabot[bot] in senzing-factory/build-resources#254
• Bump actions/download-artifact from 6 to 7 by @​dependabot[bot] in senzing-factory/build-resources#250
• Bump super-linter/super-linter from 8.3.2 to 8.4.0 by @​dependabot[bot] in senzing-factory/build-resources#255
• Bump actions/upload-artifact from 5 to 6 by @​dependabot[bot] in senzing-factory/build-resources#249
• Fix regex for filtering GitHub workflows by @​kernelsam in senzing-factory/build-resources#256

Full Changelog: senzing-factory/build-resources@v3...3.0.30

3.0.29

What's Changed

• Bump super-linter/super-linter from 8.3.0 to 8.3.1 by @​dependabot[bot] in senzing-factory/build-resources#252
• Remove validation for natural language in workflows by @​kernelsam in senzing-factory/build-resources#253

Full Changelog: senzing-factory/build-resources@v3...3.0.29

3.0.28

What's Changed

• Add Markdown formatting guidelines to PR template by @​kernelsam in senzing-factory/build-resources#244
• Enhance PR review instructions for local and GitHub by @​kernelsam in senzing-factory/build-resources#246
• 247 dockter 1 by @​docktermj in senzing-factory/build-resources#248

New Contributors

• @​docktermj made their first contribution in senzing-factory/build-resources#248

Full Changelog: senzing-factory/build-resources@v3...3.0.28

3.0.27

Full Changelog: senzing-factory/build-resources@v3...3.0.27

3.0.26

Full Changelog: senzing-factory/build-resources@v3...3.0.26

... (truncated)

Changelog

Sourced from senzing-factory/build-resources/.github/workflows/lint-workflows.yaml's changelog.

Changelog

All notable changes to this project will be documented in this file.

The format is based on Keep a Changelog, markdownlint, and this project adheres to Semantic Versioning.

[Unreleased]

• Thing 5
• Thing 4

[1.0.1] - yyyy-mm-dd

Added to 1.0.1

• Thing 3

Fixed in 1.0.1

• Thing 2

[1.0.0] - yyyy-mm-dd

Added to 1.0.0

• Thing 2
• Thing 1

Commits

• 52eb86b bump tags to v4, slack channel override, missing claude settings (#262)
• 08fb769 #260 make shared workflows generic, misc cleanup (#261)
• See full diff in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

---

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

• @dependabot rebase will rebase this PR
• @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
• @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
• @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

---

Resolves #260
Resolves #262
Resolves #261
Resolves senzing-factory/build-resources#261
Resolves senzing-factory/build-resources#257
Resolves senzing-factory/build-resources#259
Resolves senzing-factory/build-resources#254
Resolves senzing-factory/build-resources#250
Resolves senzing-factory/build-resources#255
Resolves senzing-factory/build-resources#249
Resolves senzing-factory/build-resources#256
Resolves senzing-factory/build-resources#252
Resolves senzing-factory/build-resources#253
Resolves senzing-factory/build-resources#244
Resolves senzing-factory/build-resources#246
Resolves senzing-factory/build-resources#248

[github-actions]

🤖 Claude Code Review

Code Review - Pull Request Analysis

PR Summary: Version bump for reusable workflow reference from @v3 to @v4

---

Code Quality

✅ Code follows style guide: This is a GitHub Actions workflow configuration file. The change is a simple version bump of a reusable workflow reference, which is properly formatted and follows GitHub Actions conventions.

✅ No commented-out code: None present.

✅ Meaningful variable names: Not applicable - this is a version reference change.

✅ DRY principle followed: Not applicable to this change.

✅ Identify Defects: No defects identified. This is a straightforward dependency version bump from v3 to v4 of a reusable workflow.

✅ Project memory configuration: No .claude/CLAUDE.md file exists in this project, so no project-specific requirements to verify.

Location: .github/workflows/lint-workflows.yaml:18

---

Testing

⚠️ Testing considerations:

• This change bumps a reusable workflow dependency.
• Testing should verify that the v4 workflow is compatible and functions correctly.
• Recommend confirming the v4 workflow exists and reviewing its changelog for breaking changes.
• The workflow will be tested when this PR triggers the CI pipeline.

Note: For workflow changes, manual testing isn't typically done - the PR itself will validate the workflow runs successfully.

---

Documentation

✅ Readme updated if needed: Not required for a version bump dependency change.

✅ API docs updated: Not applicable.

✅ Inline comments for complex logic: Not needed - the change is self-explanatory.

⚠️ CHANGELOG.md updated: Cannot verify if CHANGELOG.md exists or if it needs updating for this dependency bump. This type of infrastructure change may warrant a changelog entry depending on project conventions.

✅ Markdown formatting: Not applicable to this YAML file change.

---

Security

✅ No hardcoded credentials: None present.

✅ Input validation implemented: Not applicable.

✅ Proper error handling: Not applicable - this is a workflow reference.

✅ No sensitive data in logs: None present.

✅ No license files (.lic) or AQAAAD strings: None present in this change.

---

Summary

Overall Assessment: ✅ APPROVED

This is a clean, low-risk change that bumps a reusable workflow dependency version. The change is consistent with recent commits (as seen in the git history where similar bumps were made to other workflows).

Recommendations:

1. Verify the senzing-factory/build-resources/.github/workflows/lint-workflows.yaml@v4 workflow exists and review its changelog for any breaking changes or new features.
2. Consider documenting this change in CHANGELOG.md if your project tracks infrastructure updates.
3. Monitor the workflow execution on this PR to ensure v4 functions as expected.

No blocking issues found.

Automated code review analyzing defects and coding standards

[github-actions]

🤖 Claude Code Review

Code Review Analysis

✅ Code Quality

✅ Code follows style guide

• This is a GitHub Actions workflow version bump from v3 to v4
• Follows YAML formatting standards
• No style guide violations detected

✅ No commented-out code

• No commented code present

✅ Meaningful variable names

• N/A - no new variables introduced

✅ DRY principle followed

• N/A - single line change

✅ No defects identified

• Simple version bump from v3 to v4 of a reusable workflow
• No logic errors, edge cases, or security vulnerabilities introduced
• This is a dependency update only

✅ Project memory compliance

• No .claude/CLAUDE.md file found in repository
• N/A for this review

✅ Testing

⚠️ Testing considerations

• This is a workflow dependency version bump
• Testing would occur at the workflow execution level, not unit/integration tests
• Recommendation: Monitor the first workflow run after merge to ensure v4 compatibility
• No test coverage metrics applicable for workflow references

✅ Documentation

✅ README update not needed

• Dependency version bumps don't require README changes

✅ API docs not applicable

• No API changes

✅ Inline comments not needed

• Simple version reference change

⚠️ CHANGELOG.md

• No CHANGELOG.md modifications detected
• Consider documenting: "Updated lint-workflows to v4"

✅ Markdown formatting

• No markdown file changes in this PR

✅ Security

✅ No hardcoded credentials

• None present

✅ Input validation

• N/A for workflow reference

✅ Proper error handling

• Handled by the referenced workflow

✅ No sensitive data in logs

• No logging changes

✅ No license files or AQAAAD strings

• None detected

---

Summary

Overall: ✅ APPROVED with minor suggestions

This is a straightforward dependency version bump of the lint-workflows reusable workflow from v3 to v4. The change is low-risk and follows standard practices for maintaining GitHub Actions workflows.

Suggestions (non-blocking):

1. CHANGELOG.md (./.github/workflows/lint-workflows.yaml:18): Consider adding an entry documenting this version bump
2. Post-merge monitoring: Verify the v4 workflow executes successfully on the next run

No blocking issues identified. This PR is ready to merge.

Automated code review analyzing defects and coding standards

[github-actions]

🤖 Claude Code Review

⚠️ CHANGELOG.md - The CHANGELOG exists but has not been updated since 2023. This workflow version bump (infrastructure/CI change) typically wouldn't require a CHANGELOG entry in most projects, as it's not a functional change to the library itself. However, following strict process, infrastructure changes could be documented. This is a minor concern - depends on project standards.

✅ Markdown formatting - The diff doesn't contain markdown files being modified.

✅ Security

✅ No hardcoded credentials - No credentials present.

✅ Input validation implemented - N/A for this change.

✅ Proper error handling - N/A for this change.

✅ No sensitive data in logs - N/A for this change.

✅ No license files (.lic) or AQAAAD strings - None present in the diff.

---

Summary

Overall Assessment: ✅ APPROVED with minor note

This is a clean, low-risk change that bumps a reusable workflow dependency from v3 to v4. The change follows standard dependency maintenance practices.

Only Minor Note:

• .github/workflows/lint-workflows.yaml:18 - CHANGELOG.md was not updated, but this is typically acceptable for CI/CD infrastructure changes that don't affect the library's functionality. If your project standards require documenting all dependency bumps, consider adding an entry.

Recommendation: This PR can be merged. The workflow version bump appears to be part of routine maintenance (similar to other recent commits in the git history).

Automated code review analyzing defects and coding standards

[github-actions]

🤖 Claude Code Review

Code Review Analysis

Review Results

Code Quality ✅

• ✅ Code follows style guide: The change is a simple version bump in a GitHub Actions workflow file. This type of change is not covered by the code style guide, but follows proper YAML syntax and GitHub Actions conventions.
• ✅ No commented-out code: No commented code present.
• ✅ Meaningful variable names: N/A - This is a workflow reference version update.
• ✅ DRY principle followed: N/A - Single line change.
• ✅ Identify Defects: No bugs, logic errors, or security vulnerabilities introduced. The change bumps the reusable workflow version from v3 to v4.
• ⚠️ Project memory configuration: No ./.claude/CLAUDE.md file found in the repository to evaluate against.

Testing ⏭️

• N/A Unit tests: This is a GitHub Actions workflow configuration change. No application code changes require unit tests.
• N/A Integration tests: No new endpoints or features to test.
• N/A Edge cases: Not applicable for workflow version updates.
• N/A Test coverage: Not applicable for this change.

Documentation 📝

• ⚠️ Readme updated: The README.md does not document the GitHub Actions workflows or their versions. Consider if this needs documentation.
• ✅ API docs updated: N/A - No API changes.
• ✅ Inline comments: N/A - Workflow files don't require inline comments for version references.
• ❌ CHANGELOG.md updated: File .github/workflows/lint-workflows.yaml:18 - The CHANGELOG.md should be updated to reflect this dependency version bump from v3 to v4 of the lint-workflows.
• ✅ Markdown formatting: Not applicable for this YAML change.

Security 🔒

• ✅ No hardcoded credentials: No credentials present.
• ✅ Input validation: N/A - No input handling in this change.
• ✅ Proper error handling: N/A - Workflow error handling is managed by GitHub Actions.
• ✅ No sensitive data in logs: No logging changes.
• ✅ No license files: No .lic files or AQAAAD strings in this diff.

Summary

This is a low-risk change that bumps a reusable GitHub Actions workflow dependency from v3 to v4.

Required Actions:

1. ❌ CHANGELOG.md - Update the CHANGELOG.md to document this workflow dependency version bump.

Recommendations:

1. Verify that the v4 version of senzing-factory/build-resources/.github/workflows/lint-workflows.yaml is compatible with your repository's workflows and doesn't introduce breaking changes.
2. Review the release notes/changelog for the v3→v4 upgrade of the upstream workflow to understand what changes are included.

Overall Assessment: ⚠️ Approve with Changes

The code change itself is clean and simple, but the CHANGELOG.md needs to be updated before merging.

Automated code review analyzing defects and coding standards

[github-actions]

🤖 Claude Code Review

Code Review Analysis

Code Quality

✅ Code follows style guide - This is a workflow version bump, minimal code change. No style violations.

✅ No commented-out code - Clean diff with no commented code.

✅ Meaningful variable names - N/A for this change.

✅ DRY principle followed - N/A for this change.

✅ No defects identified - This is a straightforward version bump from v3 to v4 of a reusable workflow. The change is minimal and low-risk.

✅ Project memory configuration - No ./.claude/CLAUDE.md found in the repository, so N/A.

Testing

⚠️ Testing considerations - This is a GitHub Actions workflow dependency update. Testing checklist items for unit/integration tests don't apply to workflow files. However:

• The workflow should be tested by running it in the CI environment
• Verify that the v4 version of lint-workflows.yaml is compatible with the current repository setup
• Check the changelog/release notes of the upstream workflow to ensure no breaking changes

Documentation

✅ README updated - N/A for a workflow version bump unless the v4 version introduces breaking changes or new features.

✅ API docs updated - N/A for this change.

✅ Inline comments - No complex logic requiring comments.

⚠️ CHANGELOG.md - Not updated. Consider documenting this dependency version bump if your project tracks infrastructure changes. Located at root level if it exists.

✅ Markdown formatting - The workflow file is YAML, not Markdown. N/A.

Security

✅ No hardcoded credentials - None present.

✅ Input validation - N/A for this change.

✅ Proper error handling - N/A for this change.

✅ No sensitive data in logs - None present.

✅ No license files with sensitive data - Checked for .lic files and AQAAAD strings - none found in the diff.

---

Summary

Overall Assessment: ✅ APPROVED with minor recommendations

This is a low-risk change that bumps a reusable workflow dependency from v3 to v4. The change is minimal and follows best practices for dependency management.

Recommendations:

1. Verify the v4 release notes at senzing-factory/build-resources to ensure no breaking changes affect this repository
2. Consider adding this change to CHANGELOG.md if infrastructure updates are tracked
3. Monitor the first workflow run after merge to ensure v4 compatibility

File: .github/workflows/lint-workflows.yaml:18

Automated code review analyzing defects and coding standards

[github-actions]

🤖 Claude Code Review

I'll analyze this pull request diff against the Senzing code review checklist.

Code Review Analysis

Code Quality ✅

• ✅ Code follows style guide: The change is a simple version bump in a GitHub Actions workflow file. No code style issues.
• ✅ No commented-out code: None present.
• ✅ Meaningful variable names: N/A - this is a workflow configuration change.
• ✅ DRY principle followed: N/A - single line change.
• ✅ Identify Defects: No bugs, logic errors, or security vulnerabilities introduced. This is a straightforward version bump from v3 to v4 of a reusable workflow.
• ✅ Project memory configuration: No ./.claude/CLAUDE.md file concerns for this change.

Testing ⚠️

• ⚠️ Unit tests for new functions: N/A - This is a workflow configuration change, not application code.
• ⚠️ Integration tests for new endpoints: N/A
• ⚠️ Edge cases covered: N/A
• ⚠️ Test coverage > 80%: N/A - Workflow files don't have traditional test coverage.

Note: While traditional testing doesn't apply to workflow files, the change should be validated by ensuring the v4 workflow exists and is compatible.

Documentation ❓

• ❓ Readme updated if needed: Cannot determine without seeing the README and understanding if this version bump requires documentation.
• ❓ API docs updated: N/A
• ✅ Inline comments for complex logic: Not needed for this simple change.
• ❓ CHANGELOG.md updated: Should verify if this dependency version bump warrants a CHANGELOG entry. Typically, updating build tooling versions should be documented.
• ✅ Markdown files formatting: N/A - No markdown files modified.

Security ✅

• ✅ No hardcoded credentials: None present.
• ✅ Input validation implemented: N/A
• ✅ Proper error handling: N/A
• ✅ No sensitive data in logs: None present.
• ✅ No license files (.lic) or AQAAAD strings: None present.

Summary

Overall Assessment: ✅ APPROVED with minor suggestions

This is a low-risk change updating a reusable workflow dependency from v3 to v4. The change itself is clean and follows proper practices.

Action Items:

1. ❓ Verify v4 compatibility (.github/workflows/lint-workflows.yaml:18): Ensure the v4 version of senzing-factory/build-resources/.github/workflows/lint-workflows.yaml is compatible with this repository's needs and doesn't introduce breaking changes.
2. ❓ Consider CHANGELOG update: If your project documents dependency updates, consider adding an entry about updating the lint-workflows from v3 to v4.

No blocking issues found.

Automated code review analyzing defects and coding standards

[github-actions]

🤖 Claude Code Review

⚠️ No file changes detected - skipping code review.

This PR appears to contain only metadata changes (labels, description, etc.).