justinmclean commented on PR #481:
URL: https://github.com/apache/airflow-steward/pull/481#issuecomment-4678711130

   ## Correctness
   
   [advisory] Check-number inconsistency: docstring:47 + test say "#8"; section 
comment:1660 says "check #9". Reconcile (see cross-PR note).
   [advisory] validate_eval_coverage — evals_base.iterdir() / 
skills_base.iterdir() are unguarded against OSError/PermissionError; a 
restrictive CI runner would get an unhandled exception instead of a graceful 
violation. Wrap in try/except like collect_tool_python_files does.
   
   ## Security / Conventions
   No findings


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to