Skip to content

refactor: rename DataContractConfig.validate_config_update - #1843

Merged
shumkov merged 1 commit into
v1.0-devfrom
refactor/validate-config-rename
May 13, 2024
Merged

refactor: rename DataContractConfig.validate_config_update#1843
shumkov merged 1 commit into
v1.0-devfrom
refactor/validate-config-rename

Conversation

@shumkov

@shumkov shumkov commented May 10, 2024

Copy link
Copy Markdown
Collaborator

Issue being fixed or feature implemented

DataContract related structures has method validate_update to validate compartiblity as a part of Data Contract Update transition validation. For example, DocumentTypeRef.validate_update. There are more introduced in #1835. To keep consistent naming and remove repetition of meaning we need to rename DataContractConfig.validate_config_update

What was done?

  • Renamed DataContractConfig.validate_config_update to DataContractConfig.validate_update

How Has This Been Tested?

None

Breaking Changes

None

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have added "!" to the title and described breaking changes in the corresponding section if my code contains any
  • I have made corresponding changes to the documentation if needed

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@shumkov
shumkov requested a review from QuantumExplorer as a code owner May 10, 2024 17:46
@shumkov shumkov changed the title refactor: rename DataContractConfig.validate_config_update for consistency refactor: rename DataContractConfig.validate_config_update May 10, 2024
@shumkov
shumkov merged commit ed36e73 into v1.0-dev May 13, 2024
@shumkov
shumkov deleted the refactor/validate-config-rename branch May 13, 2024 07:55
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