shahar1 commented on PR #61578: URL: https://github.com/apache/airflow/pull/61578#issuecomment-3864086128
> > > Hey Shahar I asked this question in the #61365 I thought raising seperate would be good. But since now all the Headings are covered there is nothing left to do. Sorry for the inconvinence caused. From Next time onwards when I encounter such situaution I will raise all of them in a single PR. > > > > > > No problem at all! I'll try to clarify - If you make changes to multiple code parts of unrelated files, than it's better to do so in separate PRs, so if we encounter an issue in any of them - it would be easier to isolate the source of the issue. If it's only updating style of sections in docs like in this case, and the changes have the same nature - I'd prefer that it will be in a single PR because it shouldn't affect any logic inside the code. I hope that it makes sense - it's probably a matter of time and experience until you feel what is more suitable for each case. > > Yes Sure I will keep in mind this one. Thanks for always helping and suggesting me :) CI issues should have been fixed so you could merge from `main` - please note that there are issue probably related to the changes in this PR: https://github.com/apache/airflow/actions/runs/21776944806/job/62835224588?pr=61578 https://github.com/apache/airflow/actions/runs/21776944806/job/62835224593?pr=61578 -- 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]
