Skip to content

Conversation

lfgcampos
Copy link
Contributor

@lfgcampos lfgcampos commented Jul 14, 2025

  • Provided a better formatting for the README.
  • Removed a duplicated entry

Before:
image

After:
image

Provided a better formatting for the README
@daniel-kmiecik daniel-kmiecik requested a review from Copilot July 15, 2025 08:09
Copy link

@Copilot 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

A concise formatting update for the plugin README, standardizing list items and removing a duplicate parameter entry

  • Reformatted the “Since version” list to proper markdown bullets
  • Removed the duplicated defaultResponseCode entry
Comments suppressed due to low confidence (2)

modules/swagger-gradle-plugin/README.md:126

  • Remove the leading indent (4 spaces) before the list items so that the markdown bullets render properly instead of being treated as code blocks.
- Since version 2.1.6, `sortOutput` parameter is available, allowing to sort object properties and map keys alphabetically.

modules/swagger-gradle-plugin/README.md:127

  • [nitpick] Add a period at the end of this bullet for consistency and consider rephrasing to 'Since version 2.1.6, objectMapperProcessorClass also allows configuring the ObjectMapper instance used to serialize the resolved OpenAPI.' for clarity.
- Since version 2.1.6, `objectMapperProcessorClass` allows to configure also the ObjectMapper instance used to serialize the resolved OpenAPI

@daniel-kmiecik daniel-kmiecik changed the title Provide a better formatting for README docs: provide a better formatting for README Jul 16, 2025
@daniel-kmiecik daniel-kmiecik merged commit 045c8a3 into swagger-api:master Jul 16, 2025
5 checks passed
@daniel-kmiecik
Copy link
Collaborator

daniel-kmiecik commented Jul 16, 2025

@lfgcampos thank you for your contribution! We will add this to the next release.

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.

2 participants