Skip to content

Be more lenient with association end multiplicity #3373

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jul 14, 2024
Merged

Conversation

amolenaar
Copy link
Member

PR Type

What kind of change does this PR introduce?

  • Bug fix
  • Feature
  • Chore (refactoring, formatting, local variables, other cleanup)
  • Documentation content changes

What is the current behavior?

You cannot define a combination of visibilitity (and derived) and multiplicity, without providing an association end name.

What is the new behavior?

Be more lenient with what to accept. Parse visibility and derived properties too.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@amolenaar amolenaar requested a review from danyeaw July 7, 2024 16:24
@github-actions github-actions bot added the python Pull requests that update Python code label Jul 7, 2024
Copy link
Member

@danyeaw danyeaw left a comment

Choose a reason for hiding this comment

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

Another nice update, thanks!

@danyeaw danyeaw added feature A new feature and removed python Pull requests that update Python code labels Jul 14, 2024
@danyeaw danyeaw merged commit a95107e into main Jul 14, 2024
18 checks passed
@danyeaw danyeaw deleted the parse-multiplicity branch July 14, 2024 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants