Skip to content

Conversation

@bavshin-f5
Copy link
Member

@bavshin-f5 bavshin-f5 commented Oct 8, 2025

Checklist:

@bavshin-f5 bavshin-f5 requested a review from Copilot October 8, 2025 06:02
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the nginx-acme module to version 0.2.0, introducing build system improvements and a comprehensive changelog for the new release.

  • Version bump from 0.1.1 to 0.2.0 in Cargo.toml
  • Build system refactoring with parameterized module naming and separate binary/module targets
  • Addition of detailed changelog documenting breaking changes, new features, and fixes

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

File Description
Cargo.toml Version bump to 0.2.0
Makefile Build system improvements with MODULE_NAME variable and separate binary/module build targets
build/build-aws-lc.mk Refactored module path definition and reorganized variable declarations
CHANGELOG.md Added comprehensive 0.2.0 release notes with breaking changes, features, and fixes

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@bavshin-f5 bavshin-f5 requested review from ensh63 and xeioex October 8, 2025 20:42
@bavshin-f5 bavshin-f5 marked this pull request as ready for review October 8, 2025 20:42
Copy link
Contributor

@ensh63 ensh63 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

Copy link
Contributor

@xeioex xeioex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@bavshin-f5 bavshin-f5 merged commit 726b9e9 into nginx:main Oct 8, 2025
15 checks passed
@bavshin-f5 bavshin-f5 deleted the release-0.2.0 branch October 8, 2025 22:59
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.

3 participants