Skip to content

Conversation

@JonZeolla
Copy link
Member

Contributor Comments

This prevents invalid project names. Previously it would allow you to generate a project slug starting with a non alpha character which is invalid and cannot be imported in python

Pull Request Checklist

Thank you for submitting a contribution!

Please address the following items:

  • If you are adding a dependency, please explain how it was chosen.
  • If manual testing is needed in order to validate the changes, provide a testing plan and the expected results.
  • Validate that documentation is accurate and aligned to any project updates or additions.

@JonZeolla JonZeolla enabled auto-merge (squash) September 23, 2025 15:31
Copy link

@ai-coding-guardrails ai-coding-guardrails bot left a comment

Choose a reason for hiding this comment

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

I've got 1 comment for you to consider

The PR title and description are not accurate. Here are my suggestions:

Title: feat(naming): prevent invalid project names and update GitHub Actions

Description: # Contributor Comments

This PR includes two main changes:

  1. Project Name Validation: Prevents invalid project names by adding validation logic that ensures project names start with alphabetical characters and project slugs are valid Python identifiers. Previously, the system would allow generation of project slugs starting with non-alpha characters, which are invalid and cannot be imported in Python.

  2. GitHub Actions Updates: Updates checkout actions from v4 to v5 across all workflow files (ci.yml, commit.yml, release.yml, security.yml, update.yml) and updates semantic pull request validation action from v5 to v6.

Pull Request Checklist

Thank you for submitting a contribution!

Please address the following items:

  • If you are adding a dependency, please explain how it was chosen.
  • If manual testing is needed in order to validate the changes, provide a testing plan and the expected results.
  • Validate that documentation is accurate and aligned to any project updates or additions.

Reviewed with 🤟 by Zenable

@JonZeolla JonZeolla merged commit ac11444 into main Sep 23, 2025
8 checks passed
@JonZeolla JonZeolla deleted the update-actions branch September 23, 2025 15:41
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.

2 participants