Make "path_alias" module optional

I think we should, if nothing else it would be one less module to install in the testing profile.

Converted the patch to an MR against 11.x. Added deprecation for $alias_manager param.

I have an MR that is green but left a bunch of comments / questions on it. Tagging "needs architectural review" for those questions, particularly around \Drupal\system\Plugin\Condition\RequestPath and \Drupal\views\Plugin\views\argument_default\Raw.

NR as per #14

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Fixed conflict and rebased the MR, thanks!

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Updated IS with remaining tasks

Just a quick note: we're removing the hidden dependency of workspaces on path_alias in 📌 Fix Workspaces' hidden dependency on the path_alias module Needs review .

Opened 📌 Remove path_alias dependency from views Active