-
Notifications
You must be signed in to change notification settings - Fork 3
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
small typo/gitignore suggestions #4
Conversation
…bly shouldn't be checked in.
@@ -73,7 +73,7 @@ Although it omits numerous details, the following sequence diagram gives an idea | |||
|
|||
```mermaid | |||
sequenceDiagram | |||
box navy Test Suite |
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 was because the navy background doesn't render very good on github. see https://github.com/steve-bate/activitypub-testsuite/blob/main/docs/architecture.md#example-test
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.
Hmm, that's unfortunate. It looks fine when I go to that link but github supports different themes so that may be the issue. It's fine to remove it, it added a small bit of information content (grouping of the sequence actors), but it's not critical.
@@ -49,6 +49,8 @@ coverage.xml | |||
*.py,cover | |||
.hypothesis/ | |||
.pytest_cache/ | |||
/.report.json |
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.
It doesn't hurt to add this, but I usually run the tests in the server-specific project.
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.
Thanks for the typo fixes and other changes!
Hey steve. Not sure if you want any of these but figured I'd PR it to suggest.
Thanks for working on this and writing the good thoughtful docs.