Skip to content
This repository has been archived by the owner on Dec 19, 2018. It is now read-only.

Enable multiple occurring extensible directives #1377

Closed
NTaylorMullen opened this issue May 22, 2017 · 1 comment
Closed

Enable multiple occurring extensible directives #1377

NTaylorMullen opened this issue May 22, 2017 · 1 comment
Assignees
Milestone

Comments

@NTaylorMullen
Copy link
Member

A lot like the File Scoped extensible directives this issue would require directives to be top-level scoped but would allow multiple (like @inject).

@rynowak rynowak added this to the 2.0.0-preview3 milestone May 30, 2017
NTaylorMullen added a commit that referenced this issue Jun 27, 2017
- Updated existing singly occurring tests that verified that the directives must occur prior to any content to be theories that tested the joint "file scoped" behavior of the usage.
- Added tests that validate `FileScopedMultipleOccurring` can have duplicate directives without erroring.

#1377
NTaylorMullen added a commit that referenced this issue Jun 27, 2017
- The `@inject` directive must now occur prior to any markup or code due to its `DirectiveUsage.FileScopedMultipleOccurring` usage.

#1377
NTaylorMullen added a commit that referenced this issue Jun 27, 2017
- Updated existing singly occurring tests that verified that the directives must occur prior to any content to be theories that tested the joint "file scoped" behavior of the usage.
- Added tests that validate `FileScopedMultipleOccurring` can have duplicate directives without erroring.

#1377
NTaylorMullen added a commit that referenced this issue Jun 27, 2017
- The `@inject` directive must now occur prior to any markup or code due to its `DirectiveUsage.FileScopedMultipleOccurring` usage.

#1377
@NTaylorMullen
Copy link
Member Author

c2c95ec
c67e790

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants