Fix Crawler::children() to not trigger a notice for childless node
merged branch lazyhammer/issue-8048 (PR #8054)

This PR was merged into the 2.1 branch.

Discussion
----------

[DomCrawler] Fix Crawler::children() to not trigger a notice for childless node

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #8048
| License       | MIT
| Doc PR        | n/a

Commits
-------

91b8490 Fix Crawler::children() to not trigger a notice for childless node
[TwigBundle] Added missing files for phpunit
[CssSelector] Fix :nth-last-child() translation
[CssSelector] Fix :nth-last-child() translation
[Console] fix and refactor exit code handling
[Config] Adding the previous exception message into the FileLoaderLoadException so it's more easily seen

In the framework if you, for example, make a syntax error in YAML, then the true, clear message (e.g. Unable to parse at line 5 (near "framework") is nested, and harder to visually see. This includes that message in the main exception so that users can debug more easily.

Before: Cannot import resource "/path/to/app/config/config.yml" from "/path/to/app/config/config_dev.yml".

After: Cannot import resource "/path/to/app/config/config.yml" from "/path/to/app/config/config_dev.yml". (Unable to parse at line 5 (near "framework").)
merged branch lazyhammer/issue-8068-2.1 (PR #8073)

This PR was merged into the 2.1 branch.

Discussion
----------

[CssSelector] Fix :nth-last-child() translation

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #8068
| License       | MIT
| Doc PR        | n/a

Same as #8072, but for < 2.3

Commits
-------

1469953 [CssSelector] Fix :nth-last-child() translation
merged branch lazyhammer/issue-8068 (PR #8072)

This PR was merged into the 2.3 branch.

Discussion
----------

[CssSelector] Fix :nth-last-child() translation

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #8068
| License       | MIT
| Doc PR        | n/a

Commits
-------

2d9027d [CssSelector] Fix :nth-last-child() translation
Merge branch '2.1' into 2.2

* 2.1:
  [CssSelector] Fix :nth-last-child() translation
  Fix Crawler::children() to not trigger a notice for childless node
Merge branch '2.2' into 2.3

* 2.2:
  [CssSelector] Fix :nth-last-child() translation
  Fix Crawler::children() to not trigger a notice for childless node

Conflicts:
	src/Symfony/Component/CssSelector/Node/FunctionNode.php
	src/Symfony/Component/CssSelector/Tests/Node/FunctionNodeTest.php
merged branch Tobion/console-exit-code (PR #8080)

This PR was merged into the 2.1 branch.

Discussion
----------

[Console] fix and refactor exit code handling

Split of #8038

Commits
-------

5c317b7 [Console] fix and refactor exit code handling
Merge branch '2.1' into 2.2

* 2.1:
  [Console] fix and refactor exit code handling
Merge branch '2.2' into 2.3

* 2.2:
  [Console] fix and refactor exit code handling

Conflicts:
	src/Symfony/Component/Console/Application.php
merged branch weaverryan/resource-loader-show-previous-exception (PR #8089)

This PR was merged into the 2.3 branch.

Discussion
----------

[Config] Including exception message to clarify the underlying issue

Hi guys!

In the framework if you, for example, make a syntax error in YAML, then the true, clear message (e.g. Unable to parse at line 5 (near "framework") is nested, and harder to visually see. This includes that message in the main exception so that users can debug more easily.

I see this quite a bit, it's a choke point for newcomers :).

Before: Cannot import resource "/path/to/app/config/config.yml" from "/path/to/app/config/config_dev.yml".

After: Cannot import resource "/path/to/app/config/config.yml" from "/path/to/app/config/config_dev.yml". (Unable to parse at line 5 (near "framework").)

Corrections and comments warmly appreciated.

Thanks!

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | --
| License       | MIT
| Doc PR        | n/a

Commits
-------

435012f [Config] Adding the previous exception message into the FileLoaderLoadException so it's more easily seen
fix double variable assignment in ContainerDebugCommand
merged branch zakharovvi/ContainerDebugCommand_fix_double_assignment (PR #8078)

This PR was submitted for the master branch but it was merged into the 2.3 branch instead (closes #8078).

Discussion
----------

[FrameworkBundle] fix double variable assignment in ContainerDebugComman...

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| License       | MIT

Commits
-------

9d1a219 [FrameworkBundle] fix double variable assignment in ContainerDebugCommand
merged branch hason/twig_bundle (PR #8062)

This PR was merged into the 2.3 branch.

Discussion
----------

[TwigBundle] Added missing files for phpunit

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

Commits
-------

2952269 [TwigBundle] Added missing files for phpunit
[Routing] Added Router tests and fixed an exception message.
[WebProfilerBundle] Fixed composer.json