Skip to content

Build modules for NGINX 1.28.2 and 1.29.5#1040

Merged
lucacome merged 1 commit intomasterfrom
feat/nginx
Feb 27, 2026
Merged

Build modules for NGINX 1.28.2 and 1.29.5#1040
lucacome merged 1 commit intomasterfrom
feat/nginx

Conversation

@lucacome
Copy link
Copy Markdown
Member

@lucacome lucacome commented Feb 27, 2026

Summary by CodeRabbit

  • Chores
    • Updated CI/CD pipeline configuration to include additional build matrix versions for testing.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1ca25a8 and 1072dca.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

Walkthrough

Adds two new nginx versions to the CI build matrix: 1.28.2 is inserted after 1.28.1, and 1.29.5 is inserted after 1.29.4. This expands the range of nginx versions tested in the continuous integration pipeline.

Changes

Cohort / File(s) Summary
CI Build Matrix Configuration
.github/workflows/ci.yml
Added nginx_version entries 1.28.2 and 1.29.5 to the build matrix, with minor trailing-comma adjustments to maintain formatting.

Possibly related PRs

Suggested reviewers

  • viktoryamarkosyan
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and accurately describes the main change: adding two new NGINX versions (1.28.2 and 1.29.5) to the build matrix in the CI workflow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/nginx

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.

@lucacome lucacome enabled auto-merge (squash) February 27, 2026 19:19
@lucacome lucacome merged commit c132dee into master Feb 27, 2026
31 checks passed
@lucacome lucacome deleted the feat/nginx branch February 27, 2026 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant