Skip to content

fix(Caution): Validator folder for YAML or XML format #9870

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

Closed
wants to merge 2 commits into from

Conversation

Guikingone
Copy link
Contributor

Hi,

The validator waits for a config/validator folder structure if we use the "file" approach, this structure must be respected, it could be a great idea to informat that the structure is mandatory.

The validator waits for a `config/validator` folder structure if we use the "file" approach, this structure must be respected.
@javiereguiluz
Copy link
Member

@Guikingone thanks for this! Do you know which is the first Symfony version that supported this? We'll need to merge this in 2.8 or 3.4 or 4.0. Thanks!

@Guikingone
Copy link
Contributor Author

@javiereguiluz The config folder appears in the 3.4 via Flex but only 4.0 was really used via Flex.

@javiereguiluz javiereguiluz added this to the 4.0 milestone Jun 4, 2018
@javiereguiluz
Copy link
Member

Thank you @Guikingone.

javiereguiluz added a commit that referenced this pull request Jun 4, 2018
…ikingone, javiereguiluz)

This PR was submitted for the 4.1 branch but it was merged into the 4.0 branch instead (closes #9870).

Discussion
----------

fix(Caution): Validator folder for YAML or XML format

Hi,

The validator waits for a `config/validator` folder structure if we use the "file" approach, this structure must be respected, it could be a great idea to informat that the structure is mandatory.

Commits
-------

2f29f7a Reword
bc83185 fix(Caution): Validator folder for YAML or XML format
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