Re: [PR] Create additional users in breeze for testing [airflow]

[via GitHub]

aaron-wolmutt commented on PR #58126:
URL: https://github.com/apache/airflow/pull/58126#issuecomment-3513258834

   > > > A few static checks are failing.
   > > > Why is this change important?
   > > 
   > > 
   > > This change creates a user with the role op and a user with the user 
role for testing. It looks like the update breeze doc hook is failing. Do you 
know how to fix it? <img alt="image" width="884" height="399" 
src="https://private-user-images.githubusercontent.com/163314771/512214811-e752a5e6-07dd-42c9-8fcd-9e54e9d75cf0.png?jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NjI3OTE4NDcsIm5iZiI6MTc2Mjc5MTU0NywicGF0aCI6Ii8xNjMzMTQ3NzEvNTEyMjE0ODExLWU3NTJhNWU2LTA3ZGQtNDJjOS04ZmNkLTllNTRlOWQ3NWNmMC5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUxMTEwJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MTExMFQxNjE5MDdaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT0wZTA0NGJlNTNlMGU0YzUxYTAwZDUyODcxYjk3MjU0ZTI1ZGYwNmM2NGNiODIzNTc5YTI5ZTg3NDlmOTk3MGNiJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.J8rnGNHFTK8rLQldRNCeQumex8c2HP7qH6OiBecJ
 iqM">
   > 
   > Becaue it generates documentation screenshots. How to fix it is described 
where error is printed.
   > 
   > <img alt="Screenshot 2025-11-10 at 17 20 50" width="962" height="120" 
src="https://private-user-images.githubusercontent.com/595491/512265026-d6ce733f-8d85-4a42-97f7-94024b3fed7c.png?jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NjI3OTg2MzksIm5iZiI6MTc2Mjc5ODMzOSwicGF0aCI6Ii81OTU0OTEvNTEyMjY1MDI2LWQ2Y2U3MzNmLThkODUtNGE0Mi05N2Y3LTk0MDI0YjNmZWQ3Yy5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUxMTEwJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MTExMFQxODEyMTlaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT03YzFmNGI0YjVmY2ZhMTJmYzMwZmU4OWFhMjQ3YjU2YjJmYmRhZmUzMTBjZWE0MTM5ZjUyMTE5ZWU5NDY2MWQzJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.mfQrbvmomL8vc1_3CG4JeuYa4SS3cNR5zXOwgzFUlUQ";>
   > Generally you should have `prek` installed and prek hook run on your PR to 
regenerate those screenshots in your PR.
   
   I  ran ```prek --all-files``` and committed. The CI should pass now. It 
looks like ```prek --all-files``` has to be run if any breeze changes are made. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org