-
Notifications
You must be signed in to change notification settings - Fork 68
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
Tokenizer json files with arrays #43
Comments
Hi @lurumad - no, I didn't take arrays into account. Shouldn't be too hard to build it in. Otherwise, I think I may revise my thinking around tokenizing json files for web apps. You can use parameters.xml with a regex replacement expression (as opposed to an XPath expression) so in that case you can probably do what you need using parameters.xml and WebDeploy without the intervening tokenization step. Pull Requests are welcome :-P |
Great @lurumad I wanted to take some time to do it also .... but great you started it hehehe. @colindembovsky just for you to know @lurumad is a great colleague of me here in Spain, he is a master wit TS and JS :) |
hey @lurumad @lfraile I've created a branch with some corrections (and a test) for your PR. I can't figure out how to push my code to your fork, so could you pull the https://github.com/colindembovsky/cols-agent-tasks/tree/lurumad-master branch in and then update the PR? |
@lurumad @lfraile one more thing - if you run |
@colindembovsky , Hello, I've been struggling with tokenizing an array recently. In the beginning I tried to tokenize the array of strings, like appsettings.json:
} Excludes: and the following error (please, see the picture). |
@colindembovsky I'm not sure if i understand, are you planning on supporting json arrays? The other question i would wonder is if you are thinking of adding xml support? Again thanks for all of your badass work! |
Hi,
I have an autofac.json like this:
Can I tokenize modules array? connectionString and defaultSchema?
Regards
The text was updated successfully, but these errors were encountered: