Skip to content

Conversation

@MJRichardson
Copy link
Contributor

No description provided.

@MJRichardson MJRichardson self-assigned this Jun 20, 2025
@MJRichardson MJRichardson requested a review from Copilot June 20, 2025 02:23
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 prepares the project for publishing to Maven Central by updating the versioning, adding necessary metadata, and adjusting the build and deployment workflows. Key changes include increasing the version number and adding metadata in pom.xml, updating documentation in README.md to reflect Maven Central dependency information, and modifying the GitHub Actions workflow to deploy to Maven Central.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
pom.xml Updated version, added licenses/developers/scm, and new build plugins for Maven Central publishing.
README.md Updated dependency instructions and examples to reference Maven Central.
.github/workflows/maven-publish.yml Revised server configuration and added environment variables for Maven Central publishing.

<groupId>com.octopus.openfeature</groupId>
<artifactId>octopus-openfeature-provider</artifactId>
<version>0.2.0</version>
<version>0.3.0-SNAPSHOT</version>
Copy link

Copilot AI Jun 20, 2025

Choose a reason for hiding this comment

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

The version in pom.xml (0.3.0-SNAPSHOT) does not match the version referenced in README.md (0.2.0). Ensure consistency between the published artifact version and the documentation.

Copilot uses AI. Check for mistakes.
Comment on lines +25 to +30
<version>0.2.0</version> <!-- use current version number -->
</dependency>
```

```groovy
implementation group: 'com.octopus.openfeature', name: 'octopus-openfeature-provider', version: '0.2.0'
Copy link

Copilot AI Jun 20, 2025

Choose a reason for hiding this comment

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

The dependency version in the README (0.2.0) is inconsistent with the version in pom.xml (0.3.0-SNAPSHOT). Update the README to match the correct release version intended for Maven Central publishing.

Suggested change
<version>0.2.0</version> <!-- use current version number -->
</dependency>
```
```groovy
implementation group: 'com.octopus.openfeature', name: 'octopus-openfeature-provider', version: '0.2.0'
<version>0.3.0-SNAPSHOT</version> <!-- use current version number -->
</dependency>
implementation group: 'com.octopus.openfeature', name: 'octopus-openfeature-provider', version: '0.3.0-SNAPSHOT'

Copilot uses AI. Check for mistakes.
@MJRichardson MJRichardson merged commit 475f2a7 into main Jun 20, 2025
2 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.

1 participant