-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Open
Labels
EngSysThis issue is impacting the engineering system.This issue is impacting the engineering system.
Description
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
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
EngSysThis issue is impacting the engineering system.This issue is impacting the engineering system.