fix: playbook files permission mode incompatible with Windows

[wendrul]

Important

Fix file permission setting in write_file_at_user_defined_location() for Windows compatibility by adding conditional compilation.

• Behavior:
  • In write_file_at_user_defined_location() in worker.rs, added conditional compilation to handle file permission setting.
  • On Unix, uses std::os::unix::fs::PermissionsExt::from_mode() to set file permissions if mode is provided.
  • On Windows, logs an error if mode is provided, as setting file permissions is not supported.
• Misc:
  • Added #[cfg(unix)] and #[cfg(windows)] to separate Unix and Windows-specific code paths.

^{This description was created by for 01af6ce. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 01af6ce in 39 seconds

More details

• Looked at 23 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. backend/windmill-common/src/worker.rs:241

• Draft comment:
  Consider providing more context in the error message, such as suggesting alternative ways to set file permissions on Windows.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR addresses a platform-specific issue by using conditional compilation to handle file permissions differently on Unix and Windows systems. This is a good approach, but the error message on Windows could be more informative.

Workflow ID: wflow_sMiRtGweEfO23Q8m

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[cloudflare-workers-and-pages]

Deploying windmill with    Cloudflare Pages

Latest commit:	1a01dc6
Status:	✅  Deploy successful!
Preview URL:	https://5e19770e.windmill.pages.dev
Branch Preview URL:	https://win-672-fix-windows-not-acce.windmill.pages.dev

View logs