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

Make TagHelperDescriptorProvider non-legacy. #1289

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

Make TagHelperDescriptorProvider non-legacy. #1289

NTaylorMullen opened this issue May 8, 2017 · 1 comment
Assignees
@NTaylorMullen
Labels
3 - Done enhancement
Milestone
2.0.0-preview2

Comments

@NTaylorMullen
Copy link
Contributor

Rename it to TagHelperBinder and its GetTagHelperBinding => GetBinding.
@NTaylorMullen NTaylorMullen added this to the 2.0.0-preview2 milestone May 8, 2017
@NTaylorMullen NTaylorMullen self-assigned this May 8, 2017
NTaylorMullen added a commit that referenced this issue May 8, 2017
@NTaylorMullen
Rename TagHelperDescriptorProvider => TagHelperBinder.
b6ce30f 
- Moved the type out of the Legacy namespace.
- Renamed the types method to `GetBinding` since `TagHelper` is now inferred.
- Updated test names to reflect new method/class name.
- Updated product code variables to reflect new naming (provider => binder).

#1289
@NTaylorMullen NTaylorMullen mentioned this issue May 8, 2017
Merged
NTaylorMullen added a commit that referenced this issue May 8, 2017
@NTaylorMullen
Rename TagHelperDescriptorProvider => TagHelperBinder.
8bc11e9 
- Moved the type out of the Legacy namespace.
- Renamed the types method to `GetBinding` since `TagHelper` is now inferred.
- Updated test names to reflect new method/class name.
- Updated product code variables to reflect new naming (provider => binder).

#1289
NTaylorMullen added a commit that referenced this issue May 8, 2017
@NTaylorMullen
Rename TagHelperDescriptorProvider => TagHelperBinder.
774aeba 
- Moved the type out of the Legacy namespace.
- Renamed the types method to `GetBinding` since `TagHelper` is now inferred.
- Updated test names to reflect new method/class name.
- Updated product code variables to reflect new naming (provider => binder).

#1289
@NTaylorMullen
Copy link
Contributor Author

8bc11e9

@NTaylorMullen NTaylorMullen mentioned this issue May 9, 2017
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@NTaylorMullen NTaylorMullen

Labels
3 - Done enhancement
Projects
None yet
Milestone
2.0.0-preview2
Development

No branches or pull requests
1 participant
@NTaylorMullen