Skip to content

fix: 🐛 set sas_path target to mode = "always" to detect new SAS files in input_dir#233

Merged
lwjohnst86 merged 1 commit intomainfrom
fix/set-sas_paths-target-cue-to-mode-always
Feb 27, 2026
Merged

fix: 🐛 set sas_path target to mode = "always" to detect new SAS files in input_dir#233
lwjohnst86 merged 1 commit intomainfrom
fix/set-sas_paths-target-cue-to-mode-always

Conversation

@signekb
Copy link
Contributor

@signekb signekb commented Feb 27, 2026

Description

I experienced that after adding more SAS files to the input_dir, the new files weren't converted. This fixes that.
Not sure how often this would happen though?

Needs a thorough review.

Checklist

  • Ran just run-all

@signekb
Copy link
Contributor Author

signekb commented Feb 27, 2026

Another solution could be to refactor config$input_dir to include a list of SAS files instead of a dir, so the user has more control over which SAS files to convert?

Copy link
Member

@lwjohnst86 lwjohnst86 left a comment

Choose a reason for hiding this comment

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

Makes sense 👍 yea, I mean, they can control the files via the command = list_sas_files... but not sure we want to implement that.

@github-project-automation github-project-automation bot moved this from In review to Approved in Platform development Feb 27, 2026
@lwjohnst86 lwjohnst86 merged commit 23e1f5a into main Feb 27, 2026
10 checks passed
@lwjohnst86 lwjohnst86 deleted the fix/set-sas_paths-target-cue-to-mode-always branch February 27, 2026 16:24
@github-project-automation github-project-automation bot moved this from Approved to Done in Platform development Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants