gemini-code-assist[bot] commented on code in PR #152:
URL: https://github.com/apache/tvm-ffi/pull/152#discussion_r2437427727


##########
pyproject.toml:
##########
@@ -228,7 +228,7 @@ manylinux-aarch64-image = "manylinux2014"
 # will be abi3 and can be used in future versions
 # ship 314t threaded nogil version
 build = ["cp38-*", "cp39-*", "cp310-*", "cp311-*", "cp312-*", "cp314t-*"]
-skip = ["*musllinux*"]
+skip = ["*musllinux*", "cp38-macosx_*"]

Review Comment:
   ![medium](https://www.gstatic.com/codereviewagent/medium-priority.svg)
   
   For better maintainability, it would be helpful to add a comment explaining 
why the `cp38-macosx_*` build is being skipped. This will provide context for 
future developers who might wonder about this specific exclusion.



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to