Skip to content

Adding description of restrictions on creation of a new team name#62448

Merged
o-nikolas merged 4 commits into
apache:mainfrom
Srabasti:add_description_create_team
Feb 28, 2026
Merged

Adding description of restrictions on creation of a new team name#62448
o-nikolas merged 4 commits into
apache:mainfrom
Srabasti:add_description_create_team

Conversation

@Srabasti

Copy link
Copy Markdown
Contributor

This PR adds description of the requirements of a new team name when creating a new team.

Team name can contain uppercase or lowercase alphabets, digits from 0 through 9, underscore and hyphen sign. All other special characters are not allowed.

Having a prompt at onset of a new team creation will help users follow guidelines and avoid encountering errors in creation of a new team name.

Was generative AI tooling used to co-author this PR?

No

@o-nikolas o-nikolas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for submitting this!

Can you please share screenshots of the cli output for airflow teams -h and airflow teams create -h, thanks!

Comment thread airflow-core/src/airflow/cli/commands/team_command.py Outdated
@Srabasti

Copy link
Copy Markdown
Contributor Author

On running "airflow teams -h" in breeze console:
image

@Srabasti

Copy link
Copy Markdown
Contributor Author

On running "airflow teams create -h" in breeze console:

image

@Srabasti

Copy link
Copy Markdown
Contributor Author

Thanks for your review comments @o-nikolas !
Incorporated them - however the CI checks are failing due to unrelated failure https://github.com/apache/airflow/actions/runs/22424828539/job/64931470141

@Srabasti Srabasti mentioned this pull request Feb 26, 2026
@o-nikolas

Copy link
Copy Markdown
Contributor

Thanks for your review comments @o-nikolas ! Incorporated them - however the CI checks are failing due to unrelated failure https://github.com/apache/airflow/actions/runs/22424828539/job/64931470141

Thanks for incorporating those!

Btw, the screenshots you shared do not show that your changes are working, it's still the short description in both help outputs. Did you perhaps share the wrong screenshots?

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adds documentation about team name requirements to the team_create CLI command function. The documentation clarifies that team names must be 3-50 characters long and can only contain alphanumeric characters, hyphens, and underscores. This helps users understand the constraints before attempting to create a team, reducing the likelihood of errors.

Changes:

  • Updated the docstring for the team_create function to include team name validation requirements

Comment thread airflow-core/src/airflow/cli/commands/team_command.py
@Srabasti Srabasti force-pushed the add_description_create_team branch from 8d3d913 to c329108 Compare February 27, 2026 06:48
@Srabasti

Copy link
Copy Markdown
Contributor Author

Post changes, screenshots of running "airflow teams -h" and "airflow teams create -h" in breeze CLI console are as below.

image

@Srabasti Srabasti requested a review from o-nikolas February 28, 2026 05:33
@o-nikolas o-nikolas merged commit 4ac550d into apache:main Feb 28, 2026
72 checks passed
dominikhei pushed a commit to dominikhei/airflow that referenced this pull request Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants