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 fix: Add note to use .yaml for file extension #1298

Closed
wants to merge 1 commit into from

Conversation

yota-p
Copy link

@yota-p yota-p commented Jan 16, 2021

Motivation

I ran into MissingConfigException error when trying to use .yml for config, and took some time to discover that file extension is required to be .yaml.
This seems to be not written anywhere in the documentation explicitly.
So I want to add note to the document (for other beginners like me).

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

  • To pass the default ci

Related Issues and PRs

Previous discussions to support .yml or not (concluded not to support for now).
#398 #1050

The document needs to explicitly note that file extension should be `yaml`, and `.yml` can't be used. 
I ran in a error when trying to use `.yml`, and took some time to solve. 

Related issues: facebookresearch#398 facebookresearch#1050
@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 Jan 16, 2021
@omry
Copy link
Collaborator

omry commented Jan 16, 2021

Thanks for your suggestion.
The basic tutorial config file mentions that you should omit the .yaml extension, which is a big hint that the extension should be .yaml.

In addition, the intro page is not meant as a comprehensive guide. More of a minimal overview of some of the core features.
If anything, it should be in the basic tutorial.

Closing this because I think the information in the basic tutorial is sufficient. Feel free to continue the discussion here.

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