-
Notifications
You must be signed in to change notification settings - Fork 32
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
Support custom seperator in param tag. #50
base: master
Are you sure you want to change the base?
Conversation
@@ -161,8 +161,6 @@ export abstract class Block | |||
*/ | |||
public getTypeFromValue(value:string):string | |||
{ | |||
let result:Array<string>; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this isn't used in this method, that's why I deleted it.
@neild3r Could you review this, please? |
@neild3r I wonder whether you could take a look? |
The change in general seems OK but I don't know if the config name is the best description of the feature. Also shouldn't this apply to the return tag as well? |
Could you propose a name? I'm happy to hear how you'd describe that functionality.
Good point. |
I guess maybe |
No description provided.