Skip to content

[Yaml] Adds chapters for YAML lint --exclude option #14780

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

Merged
merged 1 commit into from
Sep 23, 2021

Conversation

christingruber
Copy link
Contributor

Adds the new chapters for the yaml lint --exclude and --config options.

See the feature pull: symfony/symfony#39641

@OskarStark OskarStark added this to the 5.3 milestone Jan 4, 2021
@xabbuh xabbuh modified the milestones: 5.3, next Jan 4, 2021
@christingruber christingruber force-pushed the feature/yaml-lint-config branch 2 times, most recently from 1059093 to 487176d Compare January 14, 2021 11:51
@christingruber christingruber changed the title [Yaml] Adds chapters for YAML lint --config and --exclude options [Yaml] Adds chapters for YAML lint --exclude and negatable --parse-tags option Aug 9, 2021
@christingruber christingruber force-pushed the feature/yaml-lint-config branch from 487176d to e64a77c Compare August 9, 2021 12:29
@christingruber christingruber changed the title [Yaml] Adds chapters for YAML lint --exclude and negatable --parse-tags option [Yaml] Adds chapters for YAML lint --exclude and --parse-tags option Aug 9, 2021
@christingruber christingruber changed the title [Yaml] Adds chapters for YAML lint --exclude and --parse-tags option [Yaml] Adds chapters for YAML lint --exclude option Aug 9, 2021
@christingruber christingruber force-pushed the feature/yaml-lint-config branch from e64a77c to 9b0fe51 Compare August 9, 2021 15:54
chalasr added a commit to symfony/symfony that referenced this pull request Aug 14, 2021
… to lint:yaml command (christingruber)

This PR was merged into the 5.4 branch.

Discussion
----------

[Yaml] Add --exclude and negatable --parse-tags option to lint:yaml command

| Q             | A
| ------------- | ---
| Branch?       | 5.4
| Bug fix?      | no
| New feature?  | yes
| Deprecations? | no
| Tickets       | None
| License       | MIT
| Doc PR        | symfony/symfony-docs#14780

Added a new `--exclude` option to exclude one or more specific files from multiple file list:

```lint:yaml dirname --exclude=/dirname/foo.yml --exclude=/dirname/bar.yml```

Allow negatable for the parse tags option with `--no-parse-tags` and increase the `symfony/console` dependency to version 5.3

Commits
-------

55704f3 [Yaml] Add an --exclude option to lint:yaml command
@javiereguiluz javiereguiluz added Status: Reviewed and removed Status: Needs Review Waiting Code Merge Docs for features pending to be merged labels Sep 23, 2021
@javiereguiluz javiereguiluz modified the milestones: next, 5.4 Sep 23, 2021
@javiereguiluz javiereguiluz merged commit 511613e into symfony:5.4 Sep 23, 2021
@javiereguiluz
Copy link
Member

Christin, thanks a lot for contributing this nice features and also contribute the docs for it! This was finally merged. Cheers!

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.

5 participants