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 vocabulary/activitystreams2.owl #559

Merged
merged 1 commit into from Dec 6, 2023

Conversation

ThisIsMissEm
Copy link
Contributor

@ThisIsMissEm ThisIsMissEm commented Dec 6, 2023

This file has now been moved to a separate repository located at: https://github.com/swicg/activitystreams2-owl

Closes #556 #516

This file has now been moved to a separate repository located at: https://github.com/swicg/activitystreams2-owl
@evanp evanp merged commit 359f908 into w3c:master Dec 6, 2023
@ThisIsMissEm ThisIsMissEm deleted the remove-owl-file branch December 6, 2023 21:29
@gobengo
Copy link
Contributor

gobengo commented Dec 7, 2023

In the future, please consider allowing more time for review of PRs that remove files from the repository, and consider seeking input on a w3.org SWICG list before doing so.

I disagree that this was the best way of resolving #556, which could have been fixed by correcting the owl file instead of removing it entirely.

@ThisIsMissEm
Copy link
Contributor Author

In the future, please consider allowing more time for review of PRs that remove files from the repository, and consider seeking input on a w3.org SWICG list before doing so.

I disagree that this was the best way of resolving #556, which could have been fixed by correcting the owl file instead of removing it entirely.

@evanp can correct me if I'm wrong, but I believe the understanding was that because it was non-normative and unmaintained or buggy that moving it to a place where it could be properly maintained was preferable.

Other options such as writing tooling to validate the file were also floated.

@csarven
Copy link
Member

csarven commented Dec 8, 2023

This PR doesn't resolve the underlying referenced issues. The issues are (still) relevant irrespective to the repository that the source AS2 OWL is part of. See also #516 (comment) .

As for normativeness, implementation reports or draft-extensions-policy.html , for example, are not "normative" either, but they're in this repository. (No, I'm not suggesting those should be move elsewhere or that moving them will even things out.) Point being, the whole repository is not strictly about what's deemed to be a normative reference from a specification.

The AS2 JSON-LD context fundamentally refers to the vocabulary (by identifiers) in which the activitystreams-owl was intended to provide when published. So, that should be part of the focus on resolving / maintaining what's being requested, while keeping the related material close to each other instead of further away.

@evanp
Copy link
Collaborator

evanp commented Dec 8, 2023

@gobengo I don't know if you saw that the file is now in its own repo. The idea was to give a file that we were reluctant to maintain since it would make it more official a life of its own for people who wanted to use it.

That said, yes, I shouldn't have hit the Merge button so fast. I'll revert and we can have the discussion on the list. Thanks for keeping us honest!

@nightpool
Copy link
Collaborator

nightpool commented Dec 8, 2023 via email

@evanp
Copy link
Collaborator

evanp commented Dec 8, 2023

I've reverted and will post on the email list.

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.

as:anyOf mislabeled as oneOf in OWL file?
5 participants