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

Remove references to test suite #363

Open
wants to merge 1 commit into
base: gh-pages
Choose a base branch
from
Open

Remove references to test suite #363

wants to merge 1 commit into from

Conversation

weex
Copy link

@weex weex commented Jan 15, 2023

As mentioned in #358, #351 and duplicates listed therein the test suite is no longer functional. This PR removes references to it from the README.

Copy link

@ThisIsMissEm ThisIsMissEm left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@tantek tantek left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Member

@csarven csarven left a comment

Choose a reason for hiding this comment

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

Instead of outright removal it may be more useful to acknowledge and signal that it is broken or request help. See suggestion. When the issue gets resolved, the comment can be removed.

@@ -10,6 +10,5 @@ to server API for delivering notifications and subscribing to content.

- [[https://www.w3.org/TR/activitypub/][ActivityPub's standard]]
- [[https://activitypub.rocks/][Main website]]
- [[https://test.activitypub.rocks/][Test suite]]
- [[https://activitypub.rocks/implementation-report/][Implementation Reports]] (you can generate your own using the [[https://test.activitypub.rocks/][test suite]]
- [[https://activitypub.rocks/implementation-report/][Implementation Reports]]
Copy link
Member

@csarven csarven Aug 17, 2024

Choose a reason for hiding this comment

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

Suggested change
- [[https://activitypub.rocks/implementation-report/][Implementation Reports]]
- [[https://test.activitypub.rocks/][Test suite]] (down as of 2024-08-17, see [issue](https://github.com/w3c/activitypub/issues/337))
- [[https://activitypub.rocks/implementation-report/][Implementation Reports]]

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.

4 participants