Skip to content

{CI} Refactor git hooks to improve cross-platform compatibility#8505

Merged
wangzelin007 merged 3 commits intoAzure:mainfrom
wangzelin007:wzl/fix-git-hook
Feb 25, 2025
Merged

{CI} Refactor git hooks to improve cross-platform compatibility#8505
wangzelin007 merged 3 commits intoAzure:mainfrom
wangzelin007:wzl/fix-git-hook

Conversation

@wangzelin007
Copy link
Member

Update pre-commit and pre-push hooks to support both Windows and Unix-like systems.---

This checklist is used to make sure that common guidelines for a pull request are followed.

Related command

General Guidelines

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
  • My extension version conforms to the Extension version schema

For new extensions:

About Extension Publish

There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Update pre-commit and pre-push hooks to support both Windows and Unix-like systems.
Update pre-push PowerShell script to use Read-Host for input and provide a default 'Y' response when no input is given
@azure-client-tools-bot-prd
Copy link

azure-client-tools-bot-prd bot commented Feb 25, 2025

️✔️Azure CLI Extensions Breaking Change Test
️✔️Non Breaking Changes

@azure-client-tools-bot-prd
Copy link

Hi @wangzelin007,
Please write the description of changes which can be perceived by customers into HISTORY.rst.
If you want to release a new extension version, please update the version in setup.py as well.

@azure-client-tools-bot-prd
Copy link

Hi @wangzelin007,
Since the current milestone time is less than 7 days, this pr will be reviewed in the next milestone.

@yonzhan
Copy link
Collaborator

yonzhan commented Feb 25, 2025

Thank you for your contribution! We will review the pull request and get back to you soon.

@github-actions
Copy link
Contributor

The git hooks are available for azure-cli and azure-cli-extensions repos. They could help you run required checks before creating the PR.

Please sync the latest code with latest dev branch (for azure-cli) or main branch (for azure-cli-extensions).
After that please run the following commands to enable git hooks:

pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>

@github-actions
Copy link
Contributor

@wangzelin007 wangzelin007 merged commit b1520bc into Azure:main Feb 25, 2025
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Auto-Assign Auto assign by bot CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants