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

Documentation improved. #249

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

TobiasKnauss
Copy link

I have quite some years of experience with C# and unit tests, but I have started with TwinCAT a year ago and I'm about to start with TcUnit.
While reading the documentation, I found that it lacks some useful information, so I would like to tell you about that to make life easier for others. Writing documentation is always difficult when you're the developer, because in your mind you need to find and collect all the knowledge that others would need. You might easily miss some important or helpfus info during this process.

More suggestions will follow except you'd tell me you don't want them.

Copy link
Member

@sagatowski sagatowski left a comment

Choose a reason for hiding this comment

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

  • wrong place *

@TobiasKnauss
Copy link
Author

TobiasKnauss commented Oct 4, 2024

@sagatowski Wrong place for what?

Edit:
OK, I just read the original comment that I got via E-Mail.
Since you haven't commented at another place yet: I used the markdown preview provided in GitHub. It looked correct there, so I thought this was sufficient. How could I 'compile' it?

@sagatowski
Copy link
Member

@sagatowski Wrong place for what?

Edit: OK, I just read the original comment that I got via E-Mail. Since you haven't commented at another place yet: I used the markdown preview provided in GitHub. It looked correct there, so I thought this was sufficient. How could I 'compile' it?

Sorry I left the comment in the wrong place. Now I the comments directly in the code.

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.

2 participants