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

[en] Enable the use of possessives for some nouns #2727

Merged
merged 1 commit into from
Dec 21, 2024

Conversation

ViViDboarder
Copy link
Contributor

Rather than only accepting the, many sentences should also accept a possessive like my.

This doesn't make sense for all instances of <light>, so I'll make another refactor commit after this one

Copy link
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

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

There is a merge conflict @ViViDboarder, could you take a look?

Thanks! 👍

../Frenck

@frenck frenck marked this pull request as draft December 6, 2024 20:55
@ViViDboarder
Copy link
Contributor Author

Ah. The conflict is because of #2471, which I more or less included here because the PR was listed as draft. I'll sort through and see what I come up with.

@ViViDboarder ViViDboarder marked this pull request as ready for review December 6, 2024 22:37
@tetele
Copy link
Contributor

tetele commented Dec 13, 2024

This should probably be closed in favor of #2728

@ViViDboarder
Copy link
Contributor Author

@tetele I could do that, but I split them because they deliver dependent but different changes.

I do have another conflict now though 👎

Rather than only accepting `the`, many sentences should also accept a possessive like `my`.

This doesn't make sense for all instances of `<light>`, so I'll make another refactor commit after this one
@ViViDboarder
Copy link
Contributor Author

Conflicts resolved!

Copy link
Contributor

@tetele tetele left a comment

Choose a reason for hiding this comment

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

There are some places still using stuff like a|the|my (e.g. timer intents) but we can fix that later.

This looks like a good refactoring to me

@tetele tetele merged commit ba08058 into home-assistant:main Dec 21, 2024
2 checks passed
@ViViDboarder ViViDboarder deleted the en-personal-items branch December 23, 2024 23:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants