onobc opened a new pull request, #21871:
URL: https://github.com/apache/pulsar/pull/21871

   This commit introduces the `pulsar-bom` module which fulfills PIP-326 by 
providing a Maven BOM to ease dependency management.
   
   PIP: #326
   
   ### Motivation
   As described in PIP-326, the purpose of this change is to provide a Maven 
BOM to ease dependency management for consumers of Pulsar.
   
   ### Modifications
   - Adds a new `pulsar-bom` module that provides the Maven BOM
   - The included `pom.xml` was auto-generated using the provided 
`gen-pulsar-bom.sh` script. 
   - The script can be run anytime to update the `<dependencyManagement>` 
section in the `pom.xml` w/ the latest published Pulsar modules.
   
   ### Verifying this change
   
   - [ ] Make sure that the change passes the CI checks.
   
   This change adds a Maven BOM that consumers of the Pulsar library can use. 
   As such, it is not your typical test approach to verify this. 
   I have verified manually using a local test app.
   Should I add an automated test/app that consumes the BOM? 
   If so, where would be the best place to add this in the repo?
   
   ### Does this pull request potentially affect one of the following parts:
   
   <!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
   
   *If the box was checked, please highlight the changes*
   
   - [ ] Dependencies (add or upgrade a dependency)
   - [ ] The public API
   - [ ] The schema
   - [ ] The default values of configurations
   - [ ] The threading model
   - [ ] The binary protocol
   - [ ] The REST endpoints
   - [ ] The admin CLI options
   - [ ] The metrics
   - [ x ] Anything that affects deployment (since this is a new published 
module, possibly fits this box?)
   
   ### Documentation
   
   <!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
   
   - [ ] `doc` <!-- Your PR contains doc changes. -->
   - [ ] `doc-required` <!-- Your PR changes impact docs and you will update 
later -->
   - [ x ] `doc-not-needed` <!-- Your PR changes do not impact docs -->
   - [ ] `doc-complete` <!-- Docs have been already added -->
   
   ### Matching PR in forked repository
   
   PR in forked repository: <!-- ENTER URL HERE -->
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to