YousufFFFF commented on PR #36128: URL: https://github.com/apache/superset/pull/36128#issuecomment-3560136677
Thank you for the detailed feedback. I really appreciate it. You’re absolutely right about the unrelated file changes and CI failures. To avoid adding noise and to keep the scope minimal, I created a clean branch that contains only the intended fix for the timeseries x-axis ordering. I also reran the relevant pre-commit hooks locally to ensure that everything passes for the touched file. I’m closing this PR and opening a new, clean one shortly. Thanks again for your guidance. I’m still learning the workflow and your explanation really helped me understand how to contribute properly. -- 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]
