Skip to content

Migrate update-snippets to azpysdk, require black as prereq #45007

@JennyPng

Description

@JennyPng

request:

Let's require the azpysdk sdk to run the black check before. 
We can write a message saying "Running black before as a required step". 
We can also add a --no-black if someone locally wants to save time and they are confident of what they are doing.
Why ? I was having some trouble today with black and the updater script. Running it without azpysdk and with was yielding interesting results which boiled down to:
 
I think black and the snippet updater have a relationship. Since the formatting was off, the snippet updater changes were not lining up. 
 
now when I ran black with the right config, then snippet picked up the proper changes and applied them
 
formatting plays some role in updater making changes and mapping the tags

Metadata

Metadata

Assignees

Labels

EngSysThis issue is impacting the engineering system.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions