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

win_command - add stderr_lines documentation #550

Merged
merged 3 commits into from
Sep 12, 2023

Conversation

alexdelifer
Copy link
Contributor

SUMMARY

Add the missing stderr_lines for win_command to the documentation.

For some reason this value is missing from the return values section.

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

win_command

ADDITIONAL INFORMATION

riemers/ansible-gitlab-runner#280 (comment)

We found the doc was missing while fixing a bug in the riemers.gitlab-runner role for Windows.

Notice stderr_lines was already present

ok: [srv-uitest19.zimmercas.com] => changed=false 
  cmd: c:/gitlab-runner//gitlab-runner.exe --log-format json list
  delta: '0:00:00.116974'
  end: '2023-09-03 14:39:46.779366'
  rc: 0
  start: '2023-09-03 14:39:46.662391'
  stderr: |-
    {"arch":"amd64","level":"info","msg":"Runtime platform","os":"windows","pid":4180,"revision":"d540b510","time":"2023-09-03T10:39:46-04:00","version":"15.9.1"}
    {"ConfigFile":"c:\\gitlab-runner\\config.toml","level":"info","msg":"Listing configured runners","time":"2023-09-03T10:39:46-04:00"}
  stderr_lines: <omitted>
  stdout: ''
  stdout_lines: <omitted>

@alexdelifer
Copy link
Contributor Author

Hey gang, sorry, I'm not really sure what the CI wants me to do to appease it.
If anyone can let me know what I need to change I can get this fixed :)

Thanks!

@jborean93
Copy link
Collaborator

I've manually pushed a commit to fix the yaml error as per #550 (comment). Thanks for submitting this fix.

@jborean93 jborean93 merged commit 8190c42 into ansible-collections:main Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants