Add server urls and update GitHub workflow#114
Add server urls and update GitHub workflow#114DurgaPrasad-54 wants to merge 14 commits intoPSMRI:mainfrom
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📝 WalkthroughWalkthroughThe pull request externalizes Swagger server configuration to support multiple environment-specific URLs (Dev, UAT, Demo) via environment properties, while simplifying the GitHub Actions workflow branch naming strategy from a dynamic run-based pattern to a static name. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
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. Comment |
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@src/main/java/com/iemr/inventory/config/SwaggerConfig.java`:
- Line 23: In SwaggerConfig (where Info().title("Inventory API") is built),
replace the incorrect beneficiary-focused description with an accurate Inventory
API description; update the Info().description value to something like "A
microservice for inventory management of items, stock levels, and related
operations" (or other inventory-specific wording) so the generated Swagger spec
reflects the correct service purpose.
🧹 Nitpick comments (1)
src/main/java/com/iemr/inventory/config/SwaggerConfig.java (1)
27-31: Use imports instead of inline fully-qualified class names; preferList.of()on Java 17+.
java.util.Arraysandio.swagger.v3.oas.models.servers.Serverare referenced by FQCNs inline. Since the project targets Java 17,List.of()is also a cleaner alternative toArrays.asList().Proposed refactor
Add the following imports at the top of the file:
import java.util.List; import io.swagger.v3.oas.models.servers.Server;Then simplify the server list construction:
- .servers(java.util.Arrays.asList( - new io.swagger.v3.oas.models.servers.Server().url(devUrl).description("Dev"), - new io.swagger.v3.oas.models.servers.Server().url(uatUrl).description("UAT"), - new io.swagger.v3.oas.models.servers.Server().url(demoUrl).description("Demo") - )); + .servers(List.of( + new Server().url(devUrl).description("Dev"), + new Server().url(uatUrl).description("UAT"), + new Server().url(demoUrl).description("Demo") + ));
|



Summary by CodeRabbit
New Features
Chores