developerzohaib786 commented on PR #151: URL: https://github.com/apache/polaris-tools/pull/151#issuecomment-3824364962
> Thank you for the contribution and for your interest in the apache/polaris-tools project. > > In its current form, this PR is difficult to review and merge. It contains a large number of changes that are not directly related to the stated goal, which makes the diff unnecessarily complex and increases the risk of unintended side effects. In this project, PRs expected to be focused on a single, well-defined problem. > > I recommend reducing this PR to only the changes strictly required for the intended feature or fix, and moving unrelated refactoring or experimental work to separate PRs or discussions. > > Additionally, I noticed that parts of this contribution may have been generated with the help of AI tools. While the use of AI-assisted development is not prohibited, such code often requires extra validation, refinement, and alignment with the project’s conventions and architecture before it can be considered for merging. > > Please also make sure to follow the contribution guidelines before submitting code: https://polaris.apache.org/community/contributing-guidelines/#before-you-begin-contributing-code YES i can make it possible by only focusing on only one thing (reproduceable builds) i think for this purpose pakage-lock.json and docker file would not be changed. we will open an another issue to discuss and work on other features later and i think there will be only one change in docker file which is changing of npm install command to npm ci command. and i can make this pr more short and more focused that will be easy to merge -- 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]
