search

ansible-collections / community.windows

Windows community collection for Ansible
https://galaxy.ansible.com/community/windows
GNU General Public License v3.0
199 stars 153 forks source link

Update win_scheduled_task.py #447

Closed Udayendu closed 1 year ago

Udayendu commented 1 year ago
SUMMARY
ISSUE TYPE

+label: docsite_pr

SUMMARY
ISSUE TYPE
COMPONENT NAME
ADDITIONAL INFORMATION
github-actions[bot] commented 1 year ago

Docs Build 📝

Thank you for contribution!✨

The docsite for this PR is available for download as an artifact from this run: https://github.com/ansible-collections/community.windows/actions/runs/3172105495

You can compare to the docs for the main branch here: https://ansible-collections.github.io/community.windows/branch/main

File changes:

  • M collections/community/windows/win_scheduled_task_module.html
Click to see the diff comparison. **NOTE:** only file modifications are shown here. New and deleted files are excluded. See the file list and check the published docs to see those files. ```diff diff --git a/home/runner/work/community.windows/community.windows/docsbuild/base/collections/community/windows/win_scheduled_task_module.html b/home/runner/work/community.windows/community.windows/docsbuild/head/collections/community/windows/win_scheduled_task_module.html index e971c44..b2e1313 100644 --- a/home/runner/work/community.windows/community.windows/docsbuild/base/collections/community/windows/win_scheduled_task_module.html +++ b/home/runner/work/community.windows/community.windows/docsbuild/head/collections/community/windows/win_scheduled_task_module.html @@ -883,6 +883,7 @@ - name: Disable a task that already exists community.windows.win_scheduled_task: + path: \TaskChecker name: TaskToDisable enabled: no ```
jborean93 commented 1 year ago

Is this path actually needed, the example is for a test in the root path, it's just in your situation you have a task in a sub folder needing the path option.

github-actions[bot] commented 1 year ago

This pull request is stale because it has been open for 4 weeks with no activity. Remove stale label or comment or this will be closed in 2 weeks.

Udayendu commented 1 year ago

Is this path actually needed, the example is for a test in the root path, it's just in your situation you have a task in a sub folder needing the path option.

@jborean93 In general we need to have an option for path and having things at root is always not a possibility. Based on the project needs, we should have flexibility to choose the path the way we need. Hence it would be nice to have the path as an option. So see if we can merge this PR.