tqchen edited a comment on pull request #9196:
URL: https://github.com/apache/tvm/pull/9196#issuecomment-933906489


   Thanks @hogepodge for pushing this.  A few things to note:
   
   - I noticed that there are files that are moved and lost history(e.g.  For 
example, `device_target_interactions.rst`).  It would be great to keep all the 
history (by starting from the original state, run git mv to get the files into 
the right location, then copy over the changes). 
   - There are also format that was due to possible editor auto-format 
wrapping, these are something that we can do as followups, but we also need to 
make sure one editor setting do not override another one. One thing to note is 
that it might be useful to introduce an automatic formater to ensure 
consistency as a later point.
   
   One idea is we start with a PR that only contains git mv and minimum set of 
changes to get the index right. Then we send into text polishment in followup 
smaller PRs
   
   
   


-- 
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]


Reply via email to