Skip to content

Conversation

mpritham
Copy link

Before this PR

After this PR

==COMMIT_MSG==
==COMMIT_MSG==

Possible downsides?

@changelog-app
Copy link

changelog-app bot commented Aug 12, 2025

Generate changelog in changelog/@unreleased

Type (Select exactly one)

  • Feature (Adding new functionality)
  • Improvement (Improving existing functionality)
  • Fix (Fixing an issue with existing functionality)
  • Break (Creating a new major version by breaking public APIs)
  • Deprecation (Removing functionality in a non-breaking way)
  • Migration (Automatically moving data/functionality to a new system)

Description

Add withConjureErrorParameterFormat option to WithClientOptions

Check the box to generate changelog(s)

  • Generate changelog entry

bjlaub
bjlaub previously approved these changes Aug 13, 2025
Copy link
Contributor

@bjlaub bjlaub left a comment

Choose a reason for hiding this comment

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

approving for RC

@policy-bot policy-bot bot dismissed bjlaub’s stale review August 14, 2025 15:24

Invalidated by push of 8c79e9c

@mpritham mpritham changed the title Add withConjureErrorParameterSerializationFormat option to WithClientOptions Add withConjureErrorParameterFormat option to WithClientOptions Aug 18, 2025
@changelog-app
Copy link

changelog-app bot commented Aug 18, 2025

Successfully generated changelog entry!

What happened?

Your changelog entries have been stored in the database as part of our migration to ChangelogV3.

Need to regenerate?

Simply interact with the changelog bot comment again to regenerate these entries.

@@ -3,6 +3,7 @@ apply plugin: 'org.revapi.revapi-gradle-plugin'

dependencies {
api 'com.palantir.conjure.java.api:service-config'
api 'com.palantir.conjure.java.api:errors'
Copy link
Author

Choose a reason for hiding this comment

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

This is needed because ConjureErrorParameterFormat is defined in com.palantir.conjure.java.api:errors

Comment on lines +88 to +91
- code: "java.method.removed"
old: "method T com.palantir.conjure.java.clients.ConjureClients.WithClientOptions<T>::withConjureErrorParameterSerializationFormat(com.palantir.conjure.java.api.errors.ConjureErrorParameterFormat)"
justification: "Adding method to allow clients to request error parameter serialization\
\ format"
Copy link
Contributor

Choose a reason for hiding this comment

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

This change seems weird?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants