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

Document obl:agent for English passives #1074

Merged
merged 18 commits into from
Dec 26, 2024
Merged

Document obl:agent for English passives #1074

merged 18 commits into from
Dec 26, 2024

Conversation

nschneid
Copy link
Contributor

Per UniversalDependencies/UD_English-EWT#290, English treebanks have been adopting obl:agent. This PR adds it to the English docs.

@nschneid nschneid marked this pull request as ready for review December 26, 2024 16:39
@nschneid
Copy link
Contributor Author

nschneid commented Dec 26, 2024

Also made small tweaks to the universal docs. Will leave this open for a couple days in case @dan-zeman wants to do a sanity check before I merge.

@AngledLuffa
Copy link

Does this all need a link here

https://universaldependencies.org/u/dep/obl-agent.html

or does that get automatically generated once this is merged?

@nschneid
Copy link
Contributor Author

[obl:agent]() will autolink if that's what you're asking

_u-dep/aux_.md Outdated Show resolved Hide resolved
@dan-zeman
Copy link
Member

Will leave this open for a couple days in case @dan-zeman wants to do a sanity check before I merge.

Skimmed through the changes, seems good to me (except for one comment on subject-verb agreement).

@nschneid nschneid merged commit 5831c1a into pages-source Dec 26, 2024
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.

3 participants