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

Validation of TagHelper directives should be part of the parser #945

Closed
rynowak opened this issue Jan 20, 2017 · 1 comment
Closed

Validation of TagHelper directives should be part of the parser #945

rynowak opened this issue Jan 20, 2017 · 1 comment
Assignees
@ajaybhargavb
Labels
3 - Done
Projects
Razor 2.1.0
Milestone
2.1.0-preview1

Comments

@rynowak
Copy link
Member

rynowak commented Jan 20, 2017

The taghelper handling parts of our code do a lot of slicing and dicing and validation of directives. This should be part of the parser and the nodes returned by the parser should contain all of the information we need without reparsing.
@rynowak rynowak mentioned this issue Jan 20, 2017
Closed
@Eilon Eilon added the 1 - Ready label Feb 1, 2017
@Eilon Eilon added this to the 2.0.0 milestone Feb 1, 2017
@rynowak rynowak added this to Ready in 2.0.0 Preview 1 Apr 10, 2017
@rynowak rynowak removed this from Ready in 2.0.0 Preview 1 Apr 10, 2017
@rynowak rynowak removed the Evolution label Apr 14, 2017
@rynowak rynowak added this to Ready in Razor 2.0.0 Preview 2 Apr 25, 2017
@rynowak rynowak modified the milestones: 2.0.0-preview2, 2.0.0-preview3 May 30, 2017
@rynowak rynowak removed this from Ready in Razor 2.0.0 Preview 2 May 30, 2017
@rynowak rynowak added this to Ready in Razor 2.0.0 RTM May 30, 2017
@rynowak rynowak removed this from Ready in Razor 2.0.0 RTM Jun 9, 2017
@rynowak rynowak modified the milestones: 2.1.0, 2.0.0 Jun 9, 2017
@NTaylorMullen NTaylorMullen mentioned this issue Jul 19, 2017
Closed
@rynowak rynowak added this to Ready in Razor 2.1.0 Jul 31, 2017
@rynowak rynowak assigned ajaybhargavb and unassigned NTaylorMullen Aug 1, 2017
@ajaybhargavb ajaybhargavb mentioned this issue Aug 22, 2017
Merged
@ajaybhargavb
Copy link
Contributor

fa6fde2

@ajaybhargavb ajaybhargavb moved this from Ready to Done in Razor 2.1.0 Aug 24, 2017
@NTaylorMullen NTaylorMullen mentioned this issue Oct 9, 2021
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ajaybhargavb ajaybhargavb

Labels
3 - Done
Projects
No open projects
Razor 2.1.0
Done
Milestone
2.1.0-preview1
Development

No branches or pull requests
4 participants
@Eilon @rynowak @ajaybhargavb @NTaylorMullen