Switch away from text_type shims.

 






 

 


  Review Request #11854
  —
  Created Oct. 15, 2021 and submitted  — Latest diff uploaded 

 


















 

  

   Information

  





 
 

  david
 









 
 

  Review Board
 









 
 

  master
 









 
 

  
 









 
 

  
 






 




 

  

   Reviewers

  





 
 

  reviewboard
 









 
 

  
 






 








 





  


 
 

  
In the Python 2 to 3 conversion, the name of the binary type was changed

from str to bytes, and the name of the string type was changed from

unicode to str. In addition, there were some other changes such as

the removal of the basestring type. The six library included a bunch

of helpers to ease this transition. Now that we're Python 3+ only on the

master branch, we can get rid of these and just use the builtins.

 








  


 
 

  
Ran unit tests.

 







 









 





  








 




 







   

    
    
Diff Revision 1 (Latest)



    

    


    

      
Commits

      

 

 

  
   First
   Last
  
  Summary
  ID
  Author
 



 
  
   
  
  
   
  
 
 
  
   
    

     Switch away from text_type shims.
     
In the Python 2 to 3 conversion, the name of the binary type was changed
from `str` to `bytes`, and the name of the string type was changed from
`unicode` to `str`. In addition, there were some other changes such as
the removal of the `basestring` type. The `six` library included a bunch
of helpers to ease this transition. Now that we're Python 3+ only on the
master branch, we can get rid of these and just use the builtins.

Testing Done:
Ran unit tests.

    

   
  		
  d56c52cda00badbd1d7f1f1cb7ffcdae0e4d2343
  David Trowbridge
 




    


    
    

 This diff has been split across 5 pages:
 
  <
 
 
  
   1
  
 
  
   2
  
 
  
   3
  
 
  
   4
  
 
  
   5
  
 
 
  >
 


   

  

 









 

  

   
    

     
      
      reviewboard/scmtools/hg.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/scmtools/localfile.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/scmtools/models.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/scmtools/mtn.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/scmtools/perforce.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/scmtools/plastic.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/scmtools/bzr/__init__.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/scmtools/svn/__init__.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/scmtools/svn/pysvn.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/scmtools/svn/subvertpy.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/scmtools/tests/test_bazaar.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/scmtools/tests/test_crypto_utils.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/scmtools/tests/test_git.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/scmtools/tests/test_perforce.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/scmtools/tests/test_repository_manager.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/scmtools/tests/test_svn.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/search/tests/test_search.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/site/context_processors.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/ssh/tests.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/ssh/utils.py
     
    

   
  

 





 This diff has been split across 5 pages:
 
  <
 
 
  
   1
  
 
  
   2
  
 
  
   3
  
 
  
   4
  
 
  
   5
  
 
 
  >
 







     

    

   

  

  

   
   Loading...