Skip to content
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

Progress on parsing labeled methods #1471

Merged
merged 18 commits into from Jul 5, 2019

Conversation

Projects
None yet
2 participants
@gublan24
Copy link
Member

commented Jul 5, 2019

This PR introduces labels to aspect in Umple. Aspect (before/after) will be able to add code before/after matched labels inside methods. The grammar of after/before is changed to accept an optional label ([!codeLabel:\S+:])? before the method name for an umple aspect, such the following code :
before custom SecondLabel:staticMethod { //code }

This PR has the basic implementation that enables aspect for labeled method. There are several enhancements are needed to improve the code to come. Improvement will include more test cases, more refactoring, fixing errors, etc.
** Mixsets inside methods has not been implemented yet.

@gublan24 gublan24 marked this pull request as ready for review Jul 5, 2019

@TimLethbridge TimLethbridge merged commit f6199ea into master Jul 5, 2019

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.