Skip to content

Conversation

@lwjohnst86
Copy link
Member

@lwjohnst86 lwjohnst86 commented Aug 7, 2025

Description

I removed the use of website_abbrev (it was only used once anyway). Also found a bug. We need to use realpath for dst_path otherwise it will use . if the template destination path is the same as the working directory. I also split it so the github_repo was determined from dst_path (which requires use of the realpath first).

Closes #20

This PR needs an in-depth review.

Checklist

  • Ran just run-all

@lwjohnst86 lwjohnst86 requested a review from a team as a code owner August 7, 2025 15:25
@lwjohnst86 lwjohnst86 moved this from Todo to In Review in Iteration planning Aug 7, 2025
@lwjohnst86 lwjohnst86 moved this from In Review to In Progress in Iteration planning Aug 8, 2025
@lwjohnst86 lwjohnst86 changed the title refactor: ♻️ check that website abbreviation has no spaces refactor: ♻️ revise so github_repo is name of folder Aug 8, 2025
@lwjohnst86 lwjohnst86 moved this from In Progress to In Review in Iteration planning Aug 8, 2025
Copy link
Contributor

@martonvago martonvago left a comment

Choose a reason for hiding this comment

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

Nice, just minor comments ⭐

@github-project-automation github-project-automation bot moved this from In Review to In Progress in Iteration planning Aug 8, 2025
lwjohnst86 and others added 2 commits August 8, 2025 11:48
Co-authored-by: martonvago <57952344+martonvago@users.noreply.github.com>
@lwjohnst86 lwjohnst86 requested a review from martonvago August 8, 2025 09:48
@lwjohnst86 lwjohnst86 moved this from In Progress to In Review in Iteration planning Aug 8, 2025
@github-project-automation github-project-automation bot moved this from In Review to In Progress in Iteration planning Aug 8, 2025
@lwjohnst86 lwjohnst86 merged commit a601ab6 into main Aug 8, 2025
4 checks passed
@lwjohnst86 lwjohnst86 deleted the refactor/check-that-abbrev-has-no-spaces branch August 8, 2025 12:10
@github-project-automation github-project-automation bot moved this from In Progress to Done in Iteration planning Aug 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Validator to check if abbrev has spaces and other characters?

3 participants