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

missing 'and' in docs #2808

Merged
merged 1 commit into from
Nov 30, 2023
Merged

missing 'and' in docs #2808

merged 1 commit into from
Nov 30, 2023

Conversation

EFord36
Copy link
Contributor

@EFord36 EFord36 commented Nov 30, 2023

Motivation

I was reading https://hydra.cc/docs/1.1/advanced/compose_api/#initialization-methods and there seemed to be a missing "and" in the flow of the text.

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

Docs-only change, only 'testing' needing is a maintainer to check that this is what was intended.

Related Issues and PRs

I didn't both opening an issue for this as it felt like creating more noise for the maintainers, but if it aids your workflow, I'm happy to create an issue, just let me know.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 30, 2023
@EFord36
Copy link
Contributor Author

EFord36 commented Nov 30, 2023

It looks to me like the CI failures are unrelated, so I'm not digging too hard for now, but let me know if I've mis-diagnosed!

@odelalleau
Copy link
Collaborator

Yeah, CI needs some love. Thanks for the typo fix!

@odelalleau odelalleau self-assigned this Nov 30, 2023
@odelalleau odelalleau merged commit 2f76287 into facebookresearch:main Nov 30, 2023
14 of 29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants