support colon in job filter tags scope

[HugoCasa]

Important

Update get_scope_tags in users.rs to support scope tags with colons by simplifying the logic.

• Functionality:
  • Update get_scope_tags in users.rs to support scope tags with colons by checking if the string starts with if_jobs:filter_tags: and then splitting the remaining part by commas.
• Code Simplification:
  • Simplified the logic in get_scope_tags by using starts_with and trim_start_matches instead of pattern matching with split.

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

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 8693277 in 14 seconds

More details

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

1. backend/windmill-api/src/users.rs:708

• Draft comment:
  The refactored code improves performance by avoiding unnecessary allocations. The use of starts_with and trim_start_matches is more efficient than splitting the string and collecting it into a vector.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The refactored code improves readability and performance by avoiding unnecessary allocations. The use of starts_with and trim_start_matches is more efficient than splitting the string and collecting it into a vector.

Workflow ID: wflow_DaBnqArqPhzyARKj

---

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:	8693277
Status:	✅  Deploy successful!
Preview URL:	https://7dcd75be.windmill.pages.dev
Branch Preview URL:	https://hugo-win-453-tags-if-filter.windmill.pages.dev

View logs