ashishksdev commented on PR #2926: URL: https://github.com/apache/polaris/pull/2926#issuecomment-3467804077
Thanks for the feedback and discussion on this PR! After reviewing the concerns raised, I realize this PR has several issues: 1. Author information being unintentionally changed (as @dimas-b pointed out) 2. Files showing as modifications instead of clean additions/deletions 3. Cleanup of irrelevant files already in blogs (as it is based on versioned-docs) and addition of site/content/blog is in single PR, making it difficult to review I'm going to close this PR and propose a cleaner approach: My proposed plan: - PR #1: Remove all existing content from the blogs branch (as @dimas-b preferred) - PR #2: Add the blog content (2025 folder and _index.adoc) directly to the blogs branch, preserving original author information This keeps changes separated and cleaner for review. Questions before I proceed: 1. Does this two-PR approach work for you, or would you prefer a different workflow? 2. What's the correct Git method to preserve author information when moving the blog content? Should I use git cherry-pick or another approach? 3. Should I wait for the CI discussion to conclude first, or can I proceed with clearing/adding the blog content now? Please let me know if you have any suggestions or a different approach in mind. Happy to follow whatever works best! @flyrain @dimas-b -- 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]
