Skip to content

Conversation

@JoeZiminski
Copy link
Member

@JoeZiminski JoeZiminski commented Sep 8, 2025

closes #579, this PR renames quick_validate_project function to validate_project_from_path. This is more descriptive and matches the TUI. It is a fairly straightforward find-and-replace.

No tests need to be written. There is one documentation image that needs updating, but it also needs updating for other reasons and is handled in #591

@JoeZiminski JoeZiminski linked an issue Sep 8, 2025 that may be closed by this pull request
@JoeZiminski JoeZiminski changed the title Rename 'quick_validate_project' to 'validate_project_from_path' Rename quick_validate_project to validate_project_from_path Sep 8, 2025
@JoeZiminski JoeZiminski force-pushed the rename_quick_validate_project branch 3 times, most recently from 0dfc04b to ca86d06 Compare September 8, 2025 17:04
@JoeZiminski JoeZiminski force-pushed the rename_quick_validate_project branch from ff80a64 to cbef2c5 Compare September 9, 2025 16:53
@JoeZiminski JoeZiminski requested a review from a team September 9, 2025 16:58
Copy link

@sfmig sfmig left a comment

Choose a reason for hiding this comment

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

LGTM! 😁

@JoeZiminski
Copy link
Member Author

Cheers @sfmig!

@JoeZiminski JoeZiminski merged commit 8cfdec7 into main Sep 10, 2025
17 checks passed
@JoeZiminski JoeZiminski deleted the rename_quick_validate_project branch September 10, 2025 11:50
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.

Rename quick_validate_project

2 participants