Skip to content

refactor(server): user ServerConfigBuilder to replace ServerBuilder's duplicate setter methods#1487

Merged
niklasad1 merged 2 commits intoparitytech:masterfrom
koushiro-contrib:improve-server-builder
Nov 4, 2024
Merged

refactor(server): user ServerConfigBuilder to replace ServerBuilder's duplicate setter methods#1487
niklasad1 merged 2 commits intoparitytech:masterfrom
koushiro-contrib:improve-server-builder

Conversation

@koushiro
Copy link
Contributor

@koushiro koushiro commented Oct 30, 2024

CHANGES:

  • add custom_tokio_runtime method for ServerConfigBuilder
  • add with_config constructor for ServerBuilder
  • add set_config method for ServerBuilder

BREAKING CHANGES:

  • remove all config setter methods of ServerBuilder

Close #1486

@koushiro koushiro requested a review from a team as a code owner October 30, 2024 03:52
@koushiro koushiro changed the title refactor(server): user ServerConfigBuilder to replace Builder's duplicate setter methods refactor(server): user ServerConfigBuilder to replace ServerBuilder's duplicate setter methods Oct 30, 2024
… duplicate setter methods

**CHANGES**:

- add custom_tokio_runtime method for ServerConfigBuilder
- add with_config constructor for ServerBuilder
- add set_config method for ServerBuilder

**BREAKING CHANGES**:

- remove all config setter methods of ServerBuilder

Signed-off-by: koushiro <koushiro.cqx@gmail.com>
@koushiro koushiro force-pushed the improve-server-builder branch from 1d7e0a9 to 73d7518 Compare October 30, 2024 04:56
@koushiro
Copy link
Contributor Author

@niklasad1 PTAL

@niklasad1 niklasad1 added the breaking change Breaking change in the public APIs label Oct 31, 2024
Copy link
Contributor

@niklasad1 niklasad1 left a comment

Choose a reason for hiding this comment

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

Looks good, nice

@niklasad1 niklasad1 merged commit 6177ab3 into paritytech:master Nov 4, 2024
@koushiro koushiro deleted the improve-server-builder branch November 4, 2024 13:59
@niklasad1 niklasad1 mentioned this pull request Apr 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking change Breaking change in the public APIs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

server: use ServerConfigBuilder to replace ServerBuilder's duplicate setter methods

3 participants