Skip to content

chore: 🐛 rename to file_path so write() doesn't create dir of file name - #140

Merged
lwjohnst86 merged 1 commit into
mainfrom
chore/fix-rename-var-to-file_path-in-data-write-function
Oct 29, 2026
Merged

lwjohnst86 merged 1 commit into
mainfrom
chore/fix-rename-var-to-file_path-in-data-write-function

Conversation

@signekb

@signekb signekb commented Oct 29, 2026

Copy link
Copy Markdown
Contributor

Description

When renaming the input and variable name in #134, I accidentally changed it so we call .mkdir() on the intended file name. This fixes that and change the variable name to file_path so it's clearer.

Needs a quick review.

…ir with intended file name

When renaming the the input and variable name in #134, I accidentelly changed it so we call `.mkdir()` on the intended file name.
@signekb signekb moved this from To do to In review in Platform development Oct 29, 2026
@signekb
signekb marked this pull request as ready for review October 29, 2026 15:28
@signekb
signekb requested a review from a team as a code owner October 29, 2026 15:28
@lwjohnst86 lwjohnst86 changed the title chore: 🐛 rename variable to file_path so write() doesn't create dir with intended file name chore: 🐛 rename to file_path so write() doesn't create dir of file name Oct 29, 2026
@github-project-automation github-project-automation Bot moved this from In review to Approved in Platform development Oct 29, 2026
@lwjohnst86
lwjohnst86 merged commit 56de850 into main Oct 29, 2026
6 checks passed
@lwjohnst86
lwjohnst86 deleted the chore/fix-rename-var-to-file_path-in-data-write-function branch October 29, 2026 18:49
@github-project-automation github-project-automation Bot moved this from Approved to Done in Platform development Oct 29, 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