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

89 more details in the protocol #97

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

FlinkbaumFAU
Copy link
Collaborator

No description provided.

@FlinkbaumFAU FlinkbaumFAU linked an issue Jul 16, 2022 that may be closed by this pull request
@FlinkbaumFAU
Copy link
Collaborator Author

Unfortunately the test on windows-latest fails. This happens when the logfile is being opened in src/extended-protocol/extended-protocol.service.ts line 42.
It is a simple command to open a file yet it fails on windows.
I do not have a windows installation available and thus I have a hard time debugging it.
@RumiAust maybe you can help? Can you have a look at it?

Copy link
Collaborator

@RumiAust RumiAust left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks fine, the test case is failing of type error (probably undefined need to be initialized or the.html file. I tried to write test cases and check but was not working (commented currently). @FlinkbaumFAU can have a look at the test cases.

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

Successfully merging this pull request may close these issues.

More details in the protocol
3 participants