Yicong-Huang commented on code in PR #5692: URL: https://github.com/apache/texera/pull/5692#discussion_r3416025237
########## amber/dev-requirements.txt: ########## @@ -20,6 +20,12 @@ # pip-licenses output and never need to be tracked in LICENSE-binary # / NOTICE-binary. Not installed by packaging. +# pytest test runner and plugins. Pytest's INI/TOML config parser +# (iniconfig) is pulled in as a transitive — no need to pin it here. Review Comment: I think we actually don't need to declare transitive dependencies, even for comments. I will remove the comment if no objection? -- 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]
