-
Notifications
You must be signed in to change notification settings - Fork 0
Configure reposilite repository #43
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Configure reposilite repository #43
Conversation
There was a problem hiding this 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 configures a custom Reposilite Maven repository for the project by adding distribution management and repository definitions to both the root POM and all module POMs.
- Adds Reposilite repository configuration for both releases and snapshots
- Propagates the same repository settings to all Maven modules in the project
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| pom.xml | Adds root-level Reposilite distribution management and repository configuration |
| phoenixd-test/pom.xml | Duplicates Reposilite configuration in test module |
| phoenixd-rest/pom.xml | Duplicates Reposilite configuration in REST module |
| phoenixd-model/pom.xml | Duplicates Reposilite configuration in model module |
| phoenixd-base/pom.xml | Duplicates Reposilite configuration in base module |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| <id>reposilite-releases</id> | ||
| <url>https://maven.398ja.xyz/releases</url> | ||
| </repository> | ||
| </repositories> |
Copilot
AI
Aug 20, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The distributionManagement and repositories sections are duplicated across all module POMs. Since these modules inherit from the root POM, this configuration should be inherited automatically and doesn't need to be redefined in each module.
| </repositories> |
| <id>reposilite-releases</id> | ||
| <url>https://maven.398ja.xyz/releases</url> | ||
| </repository> | ||
| </repositories> |
Copilot
AI
Aug 20, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The distributionManagement and repositories sections are duplicated across all module POMs. Since these modules inherit from the root POM, this configuration should be inherited automatically and doesn't need to be redefined in each module.
| </repositories> |
| <id>reposilite-releases</id> | ||
| <url>https://maven.398ja.xyz/releases</url> | ||
| </repository> | ||
| </repositories> |
Copilot
AI
Aug 20, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The distributionManagement and repositories sections are duplicated across all module POMs. Since these modules inherit from the root POM, this configuration should be inherited automatically and doesn't need to be redefined in each module.
| </repositories> |
| <id>reposilite-releases</id> | ||
| <url>https://maven.398ja.xyz/releases</url> | ||
| </repository> | ||
| </repositories> |
Copilot
AI
Aug 20, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The distributionManagement and repositories sections are duplicated across all module POMs. Since these modules inherit from the root POM, this configuration should be inherited automatically and doesn't need to be redefined in each module.
| </repositories> |
Summary
Testing
mvn -q verify(fails: CreateInvoiceParamTest, PayBolt11InvoiceInvoiceResponseTest, PayLightningAddressInvoiceResponseTest)https://chatgpt.com/codex/tasks/task_b_68a64afb04a08331b2bd9fdf8020c3d9