Skip to content

correct flow identification syntax#206

Merged
raphael-goetz merged 2 commits intomainfrom
#205-cron-fix
Apr 19, 2026
Merged

correct flow identification syntax#206
raphael-goetz merged 2 commits intomainfrom
#205-cron-fix

Conversation

@raphael-goetz
Copy link
Copy Markdown
Member

Resolves: #205

Copilot AI review requested due to automatic review settings April 19, 2026 16:49
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 19, 2026

GitLab Pipeline Action

General information

Link to pipeline: https://gitlab.com/code0-tech/development/draco/-/pipelines/2463723431

Status: Passed
Duration: 2 minutes

Job summaries

docs:preview

Documentation preview available at https://code0-tech.gitlab.io/-/development/telescopium/-/jobs/13989069162/artifacts/out/index.html

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Cron adapter’s flow lookup pattern to correctly match CRON flows (per #205) and adds additional debug logging to help diagnose flow identification behavior.

Changes:

  • Fix cron flow key pattern from CRON.< to CRON.* so flow lookup matches expected wildcard semantics.
  • Add debug logs around cron flow identification and computed cron configuration.
  • Refactor the final boolean match check into an intermediate is_match variable with logging.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread adapter/cron/src/main.rs Outdated
Comment thread adapter/cron/src/main.rs
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Raphael Götz <52959657+raphael-goetz@users.noreply.github.com>
@raphael-goetz raphael-goetz merged commit 7da396b into main Apr 19, 2026
5 checks passed
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.

Fix Cron Adapter Flow Identification

2 participants