Skip to content

Conversation

@dadangnh
Copy link
Contributor

@dadangnh dadangnh commented Oct 17, 2025

What this PR does / why we need it

This PR updates the n8n Helm chart and application versions to 1.0.16 and 1.116.2 respectively.

Which issue this PR fixes

Special notes for your reviewer

Checklist

Please place an 'x' in all applicable fields and remove unrelated items.

Version and Documentation

Testing and Validation

  • Ran ah lint locally without errors
  • Ran Chart-Testing: ct lint --chart-dirs charts/n8n --charts charts/n8n --validate-maintainers=false
  • Tested chart installation locally
  • Tested with example configurations in /examples directory

Summary by CodeRabbit

Summary by CodeRabbit

  • Chores
    • Updated application version to 1.116.2, delivering the latest upstream fixes, improvements, and metadata updates.
    • Adjusted CI/lint workflow configuration to use a specific schema validation tool version to stabilize chart validation steps.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 17, 2025

Walkthrough

Bumps Helm chart metadata: charts/n8n/Chart.yaml chart version 1.0.15 → 1.0.16, appVersion 1.112.0 → 1.116.2, and updates the artifacthub.io/changes annotation; adds yamale_version: "6.0.0" input to the chart-testing step in .github/workflows/lint-test.yaml.

Changes

Cohort / File(s) Summary
Chart metadata bump
charts/n8n/Chart.yaml
Chart version: 1.0.151.0.16; appVersion: 1.112.01.116.2; artifacthub.io/changes description updated to reference 1.116.2.
CI workflow input
.github/workflows/lint-test.yaml
Adds yamale_version: "6.0.0" as an input to the chart-testing setup step (no control-flow or step changes).

Sequence Diagram(s)

(omitted — change is metadata/version bump and a CI input; no runtime/control-flow changes introduced)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Areas to pay extra attention:

  • charts/n8n/Chart.yaml — ensure version/appVersion consistency and annotation formatting.
  • .github/workflows/lint-test.yaml — confirm the added yamale_version input is compatible with the chart-testing action version used.

Possibly related PRs

Suggested reviewers

  • Vad1mo
  • RoseSecurity

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The pull request title "chore: bump n8n chart and app versions to 1.0.16 and 1.116.2" directly and clearly describes the main objective stated in the PR description: updating the n8n Helm chart version to 1.0.16 and the n8n application version to 1.116.2. The title is specific, concise, and uses concrete version numbers rather than vague language. While the PR also includes a workflow change (adding yamale_version to lint-test.yaml), this appears to be a supporting fix for a linting issue rather than the primary objective, so its omission from the title is appropriate and acceptable per the check guidelines.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e645a5c and a957573.

📒 Files selected for processing (1)
  • .github/workflows/lint-test.yaml (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: lint-test
🔇 Additional comments (1)
.github/workflows/lint-test.yaml (1)

35-36: Verify that yamale 6.0.0 resolves the reported linter failure.

The added yamale_version: "6.0.0" input pins the yamale version to address the linter failure mentioned in PR comments. Please confirm that:

  1. This version actually resolves the linter failure when run.
  2. This version is compatible with helm/[email protected].
  3. Whether this is a permanent fix or a temporary workaround pending an upstream fix.

Additionally, clarify if this version pinning should be permanent or if the action will eventually default to a compatible version that doesn't require explicit pinning.


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 and usage tips.

@dadangnh dadangnh changed the title chore: bump n8n chart and app versions to 1.0.16 and 1.115.3 chore: bump n8n chart and app versions to 1.0.16 and 1.116.2 Oct 27, 2025
@Vad1mo
Copy link
Member

Vad1mo commented Oct 27, 2025

@dadangnh linter fails

@dadangnh
Copy link
Contributor Author

@dadangnh linter fails

Hi @Vad1mo , the issue seems from the GH action upstream, I'll try to bump the yamale version and see if it's help

@dadangnh
Copy link
Contributor Author

All green @Vad1mo 🎉

@Vad1mo Vad1mo merged commit e2b5196 into 8gears:main Oct 27, 2025
4 checks passed
@dadangnh dadangnh deleted the chore/update-n8n-app-to-v1.115.3 branch October 28, 2025 00:48
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