Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sync eng/common directory with azure-sdk-tools for PR 1421 #1616

Merged
merged 6 commits into from
Feb 23, 2021

Conversation

azure-sdk
Copy link
Collaborator

Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#1421 See eng/common workflow

… Eliminate default base branch "master" as this will be changed later, providing no default and using a mandatory parameter means a dev must provide the value * Check for existence of $CspellConfigPath * No need to Set-Location if run from the root of the repo (most common scenario) * -join
…Add reference to spelling docs, exit 0, Rename to check-spelling-in-changed-files.ps1,
… validation farther down that will also catch these errors), Update comments and script name
@azure-sdk azure-sdk requested review from danieljurek and a team as code owners February 22, 2021 21:51
@azure-sdk azure-sdk removed the request for review from a team February 22, 2021 21:51
@azure-sdk azure-sdk added Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system. labels Feb 22, 2021
@azure-sdk azure-sdk merged commit 383d1f8 into master Feb 23, 2021
@azure-sdk azure-sdk deleted the sync-eng/common-spellcheck-1421 branch February 23, 2021 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants