Skip to content
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

update quiet mode documentation to reflect actual behavior #4373

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rishichawda
Copy link
Member

@rishichawda rishichawda commented Jan 28, 2025

Description

This is related to the change that we did to fix quiet mode in chef/chef#14779 (backported to chef 18 in chef/chef#14817).

In short, quiet mode wasn't working before. So with the fix we didn't want to break anyone's configuration. Hence, quiet mode was updated to be set to false by default. The documentation was also a little confusing since quiet mode is supposed to suppress the output completely vs "controlling verbosity".

Definition of Done

Issues Resolved

https://progresssoftware.atlassian.net/browse/CHEF-9161

Related PRs

Check List

  • Spell Check
  • Local build
  • Examine the local build
  • All tests pass

this is related to the change that we did to fix quiet mode in chef/chef#14779. in brief, quiet mode wasn't working before. so with the fix we didn't want to break anyone's cofiguration. so quiet mode was changed to be set to false by default. the documentation was also a little confusing since quiet mode is supposed to suppress the output completely vs "controlling verbosity".

Signed-off-by: rishichawda <[email protected]>
@rishichawda rishichawda requested a review from a team as a code owner January 28, 2025 13:29
Copy link

netlify bot commented Jan 28, 2025

Deploy Preview for chef-web-docs ready!

Name Link
🔨 Latest commit 1520783
🔍 Latest deploy log https://app.netlify.com/sites/chef-web-docs/deploys/6798dbd252fa600008f2c87b
😎 Deploy Preview https://deploy-preview-4373--chef-web-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@IanMadd
Copy link
Contributor

IanMadd commented Jan 30, 2025

This change applies to Client 19 and 18.7.x.

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