Skip to content

enforceExistence: fixup enforce existance checking#22

Merged
qfox merged 1 commit intomasterfrom
hotfix/nested-jsdocs-existance
Nov 2, 2014
Merged

enforceExistence: fixup enforce existance checking#22
qfox merged 1 commit intomasterfrom
hotfix/nested-jsdocs-existance

Conversation

@qfox
Copy link
Copy Markdown
Member

@qfox qfox commented Nov 2, 2014

  • add tests for nested methods
  • add tests for variable expressions (aka privates)
  • now rule checking not only node.id (function name)

Fixes #21

@qfox qfox force-pushed the hotfix/nested-jsdocs-existance branch from ddcbed4 to 538c1cf Compare November 2, 2014 09:40
qfox pushed a commit that referenced this pull request Nov 2, 2014
- add tests for nested methods
- add tests for variable expressions (aka privates)
- now rule checking not only node.id (function name)

Fixes #21
Closes #22
@qfox qfox changed the title enforceExistance: fixup enforce existance checking enforceExistence: fixup enforce existance checking Nov 2, 2014
- add tests for nested methods
- add tests for variable expressions (aka privates)
- add tests for objects properties
- now rule checking not only node.id (function name)

Fixes #21
Closes #22
@qfox qfox force-pushed the hotfix/nested-jsdocs-existance branch from 538c1cf to 298dd2e Compare November 2, 2014 19:00
qfox added a commit that referenced this pull request Nov 2, 2014
enforceExistence: fixup enforce existance checking
@qfox qfox merged commit eba5fc2 into master Nov 2, 2014
@qfox qfox deleted the hotfix/nested-jsdocs-existance branch November 2, 2014 19:04
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.

enforceExistence "overlooks" certain method definitions

1 participant