Skip to content
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

The string is missing the terminator: "@. #18

Closed
pete076 opened this issue Aug 10, 2023 · 7 comments
Closed

The string is missing the terminator: "@. #18

pete076 opened this issue Aug 10, 2023 · 7 comments

Comments

@pete076
Copy link

pete076 commented Aug 10, 2023

When I run your script I get the following error:

image

After al little bit of digging, when i remove this part from 'LogicAp.Doc.ps1' the script runs fine but no graphic is embedded in the resulting .md file

image

How can this be solved?

@stefanstranger
Copy link
Owner

stefanstranger commented Aug 10, 2023

Hi @pete076

Can you enable verbose logging and share the verbose output too?
Just add at the end -verbose

Yes if you remove that the mermaid diagram will not be shown.

Just set a breakpoint on line 214. Check if the mermaid diagram object is created correctly.

image

Which version are you using?

@pete076
Copy link
Author

pete076 commented Aug 10, 2023

Mermaid code is generated well.

verbose logfile.txt
content mermaid.txt

Im running this within VSCode version 1.81
PowerShell Extension v2023.6.0

It makes no different if I run this outside VSCode -> this is not entirely true.
Because I started within VS Code I had to make some changes to your script:

I've added the following variables:
$FilePath = ""
$ScriptRoot = "C:\testfolder\logicappdocs-main\src"

And changed the following lines
image

When running outside VS Code, these changes aren't needed

When running outside VS Code I get some more errors as a result from the first error.

image

@pete076
Copy link
Author

pete076 commented Aug 10, 2023

Solution found:

file Logic-App.Doc.ps1

image

#25 and #27 contained 7 'backtic's', this should be 6

@stefanstranger
Copy link
Owner

stefanstranger commented Aug 10, 2023

I meant the version of logicappsdocs 😁

@pete076
Copy link
Author

pete076 commented Aug 10, 2023

Author: Stefan Stranger
Github: https://github.com/stefanstranger/logicappdocs
Version: 1.1.0

Main branche, downloaded today

@stefanstranger
Copy link
Owner

stefanstranger commented Aug 11, 2023

Hi @pete076 ,

When I look at the LogicApp.Doc.ps1 script on lines 24 till 28 I see that it there is a here string configured.

It contains 7 backtic's at the moment and you are saying it needs to be 6 and then it works for you?

Strange because this has not given me any errors until now. Let me validate this.

Regards,
Stefan

@stefanstranger
Copy link
Owner

After testing with 6 backtics it works fine. Fixed in development branch powerautomatissues. Will be available in next release.

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

No branches or pull requests

2 participants