Skip to content

Conversation

@kalindi-adhiya
Copy link
Contributor

Motivation

Fixes #NNN

Proposed changes

Alternatives considered

Testing steps

  1. Follow the contribution guide to set up your development environment or download a pre-built acli.phar for this PR.
  2. If running from source, clear the kernel cache to pick up new and changed commands: ./bin/acli ckc
  3. Check for regressions: (add specific steps for this pr)
  4. Check new functionality: (add specific steps for this pr)

@codecov
Copy link

codecov bot commented Jan 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.27%. Comparing base (866ef61) to head (ec60f1f).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #1956   +/-   ##
=========================================
  Coverage     92.27%   92.27%           
  Complexity     1916     1916           
=========================================
  Files           122      122           
  Lines          6993     6994    +1     
=========================================
+ Hits           6453     6454    +1     
  Misses          540      540           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions
Copy link

Try the dev build for this PR: https://acquia-cli.s3.amazonaws.com/build/pr/1956/acli.phar

curl -OL https://acquia-cli.s3.amazonaws.com/build/pr/1956/acli.phar
chmod +x acli.phar

@kalindi-adhiya kalindi-adhiya marked this pull request as ready for review January 20, 2026 05:59
Copilot AI review requested due to automatic review settings January 20, 2026 05:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a documentation link to the authentication error message that appears when users encounter federated authentication issues. This helps users quickly access known issue documentation for troubleshooting.

Changes:

  • Added a help message with a documentation URL pointing to the known issues page for federated authentication
  • Updated the test to verify the new help message is included in the error output

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/EventListener/ExceptionListener.php Adds documentation link to the federated authentication error help messages
tests/phpunit/src/Misc/ExceptionListenerTest.php Updates test expectation to include the new documentation link message

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kalindi-adhiya kalindi-adhiya merged commit 37ace51 into acquia:main Jan 21, 2026
26 checks passed
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.

2 participants