Skip to content

Validate data in PCM Builder#122

Merged
sebinside merged 1 commit intomainfrom
validation
Jan 30, 2024
Merged

Validate data in PCM Builder#122
sebinside merged 1 commit intomainfrom
validation

Conversation

@Entenwilli
Copy link
Member

This PR uses the validation logic from previous PRs and actually calls the behavior. This PR closes #118

@Entenwilli Entenwilli added bug Something isn't working core Related to the core DFD/PCM data flow analysis labels Jan 29, 2024
@Entenwilli Entenwilli added this to the 2.0.0 milestone Jan 29, 2024
@Entenwilli Entenwilli requested a review from sebinside January 29, 2024 20:38
@Entenwilli Entenwilli self-assigned this Jan 29, 2024
Copy link
Member

@sebinside sebinside left a comment

Choose a reason for hiding this comment

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

LGNTM

@sebinside sebinside merged commit 3fe84ba into main Jan 30, 2024
@sebinside sebinside deleted the validation branch January 30, 2024 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working core Related to the core DFD/PCM data flow analysis

Projects

None yet

Development

Successfully merging this pull request may close these issues.

validate() is not called in the analysis builder

2 participants