Skip to content

[smart_holder] git merge master #3613

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 11 commits into from
Jan 12, 2022
Merged

[smart_holder] git merge master #3613

merged 11 commits into from
Jan 12, 2022

Conversation

rwgk
Copy link
Collaborator

@rwgk rwgk commented Jan 12, 2022

Description

Helper/scratch PR for testing.

Suggested changelog entry:



    

  
  



      


          

          
  

    
  
    
    

  

  



          

        
      




  





       
            



      

  
        

    

  
  

  
          dependabot
  bot
and others
        added 11 commits
        January 10, 2022 10:29


  

      

  
  

  
          

  

    

      


  

      
        @dependabot
  




      

        
          chore(deps): bump pypa/gh-action-pypi-publish from 1.4.2 to 1.5.0 (py…
        

          
                        
      


      

        

    
    
    

  

    


      


      

        

          

    
    
    

  

  

    
  



        

      


      

        
          fb9a222
        
      

    

  

    

      
…bind#3606)

Bumps [pypa/gh-action-pypi-publish](https://github.com/pypa/gh-action-pypi-publish) from 1.4.2 to 1.5.0.
- [Release notes](https://github.com/pypa/gh-action-pypi-publish/releases)
- [Commits](pypa/gh-action-pypi-publish@v1.4.2...v1.5.0)

---
updated-dependencies:
- dependency-name: pypa/gh-action-pypi-publish
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

    







      

  
  

  
          

  

    

      


  

      
        @pre-commit-ci
  




      

        
          [pre-commit.ci] pre-commit autoupdate (pybind#3609)
        

          
                        
      


      

        

    
    
    

  

    


      


      

        

          

    
    
    

  

  

    
  



        

      


      

        
          b66328b
        
      

    

  

    

      
updates:
- [github.com/pre-commit/mirrors-mypy: v0.930 → v0.931](pre-commit/mirrors-mypy@v0.930...v0.931)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

    







      

  
  

  
          

  

    

      


  

      
        @Skylion007
  




      

        
          Add additional info to TypeError when C++->Python casting fails (pybi…
        

          
                        
      


      

        

    
    
    

  

    


      


      

        

          

    
    
    

  

  

    
  



        

      


      

        
          ef070f7
        
      

    

  

    

      
…nd#3605)

* Add additional info to TypeInfo when C++->Python casting fails

* Fix typo

* Address reviewer comments

    







      

  
  

  
          

  

    

      


  

      
        @rwgk
      
        @ax3l
  




      

        
          Replacing ICC C++14 with C++17 (pybind#3570)
        

          
                        
      


      

        

    
    
    

  

    


      


      

        

          

    
    
    

  

  

    
  



        

      


      

        
          f588810
        
      

    

  

    

      
* Replacing ICC C++14 with C++17

* ICPC: -diag-disable:conversion

Try to suppress the `-Werror-all` promotion in `#include <variant>`

* Trying `-DPYBIND11_WERROR=OFF`

* Trying `-Wno-conversion` (todo: make specific to C++17 if it works)

* Inserting -Wno-conversion only for C++17

Co-authored-by: Axel Huebl <[email protected]>

    







      

  
  

  
          

  

    

      


  

      
        @Skylion007
  




      

        
          (chore): Remove deprecated c-headers (pybind#3610)
        

          
                        
      


      

        

    
    
    

  

    


      


      

        

          

    
    
    

  

  

    
  



        

      


      

        
          d434b5f
        
      

    

  

    

      
* Remove deprecated c-headers

* Update calls to old cfunctions

* Add missing one

* Add another missing one

    







      

  
  

  
          

  

    

      


  

      
        @iwanders
      
        @pre-commit-ci
  




      

        
          A way to register additional test targets and support .py only tests. (…
        

          
                        
      


      

        

    
    
    

  

    


      


      

        

          

    
    
    

  

  

    
  



        

      


      

        
          21911e1
        
      

    

  

    

      
…pybind#3590)

* A way to register additional test targets.

* Support specifying tests with extension.

* Ensure TEST_OVERRIDE is backwards compatible.

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Ensure regex is non greedy.

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

    







      

  
  

  
          

  

    

      


  

      
        @Skylion007
  




      

        
          Add clang-tidy readability checks for sus args (pybind#3611)
        

      


      

        

    
    
    

  

    


      


      

        

          

    
    
    

  

  

    
  



        

      


      

        
          f8d4aa4
        
      

    

  







      

  
  

  
          

  

    

      


  

      
        @rwgk
  




      

        
          Temporarily backing out all test_class_sh, test_classh additions.
        

      


      

        

    
    
    

  

      


      

        

          

    
    
    

  

  

    
  



        

      


      

        
          ea4cc14
        
      

    

  







      

  
  

  
          

  

    

      


  

      
        @rwgk
  




      

        
          Merge branch 'master' into sh_merge_master
        

      


      

        

    
    
    

  

      


      

        

          

    
    
    

  

  

    
  



        

      


      

        
          ea0c1e7
        
      

    

  







      

  
  

  
          

  

    

      


  

      
        @rwgk
  




      

        
          Adding back test_class_sh, test_classh additions, building on PR pybi…
        

          
                        
      


      

        

    
    
    

  

      


      

        

          

    
    
    

  

  

    
  



        

      


      

        
          aa209cc
        
      

    

  

    

      
…nd#3590.

    







      

  
  

  
          

  

    

      


  

      
        @rwgk
  




      

        
          Tracking ci.yml changes from master.
        

      


      

        

    
    
    

  

      


      

        

          

    
    
    

  

  

    
  



        

      


      

        
          d6c174c
        
      

    

  







  








      

  
          

  

    
  


  

      

    @rwgk
rwgk



    mentioned this pull request
    
      Jan 12, 2022
    





  


  

      
   Merged

  







  









      

  
      



  

  @rwgk





  


    

      

  

    

        
    
    

  


      
          
  
      
            Copy link

  

      
    

  


  

      

  
    Collaborator


      

  Author


  


  

    

      

      
            rwgk
  

      

      

      commented


        Jan 12, 2022


      
    


  





      


        


  
    

      
          
Closing and re-opening a few seconds later to re-trigger the testing.


Last night there were many error of this kind:


Download action repository 'actions/checkout@v2' (SHA:ec3a7ce113134d7a93b817d10a8272cb61118579)
Warning: Failed to download action 'https://api.github.com/repos/actions/checkout/tarball/ec3a7ce113134d7a93b817d10a8272cb61118579'. Error: Response status code does not indicate success: 503 (Service Unavailable).
Warning: Back off 15.548 seconds before retry.
Warning: Failed to download action 'https://api.github.com/repos/actions/checkout/tarball/ec3a7ce113134d7a93b817d10a8272cb61118579'. Error: Response status code does not indicate success: 502 (Bad Gateway).
Warning: Back off 19.778 seconds before retry.
Error: Response status code does not indicate success: 503 (Service Unavailable).


      		
    

  





        


            

            
            

              
            

        

      


      

            
  

    
  
    
    

  

  


    












      

  
            

    

      
    

    


      

          @rwgk
rwgk




            closed this


      Jan 12, 2022

    

  


    


    

    

      
    

    


      

          @rwgk
rwgk




        reopened this

      Jan 12, 2022

    

  








      

  
      



  

  @rwgk





  


    

      

  

    

        
    
    

  


      
          
  
      
            Copy link

  

      
    

  


  

      

  
    Collaborator


      

  Author


  


  

    

      

      
            rwgk
  

      

      

      commented


        Jan 12, 2022


      
    


  





      


        


  
    

      
          
This PR was also run through the Google global testing (2022-01-12 00:00 batch).

      		
    

  





        


            

            
            

              
            

        

      


      

            
  

    
  
    
    

  

  


    












      

  
          

  

    
  

  



        @rwgk
  rwgk



        merged commit d6c174c
      into

      
  
    pybind:smart_holder
  


    Jan 12, 2022

  







  

  
  

  
      @rwgk
  rwgk


    
    deleted the
    
      
        sh_merge_master
    
    branch

    January 12, 2022 16:55    
    




    

    

      
    

    


      

          @github-actions
github-actions
bot



          added
  the 

  needs changelog

  Possibly needs a changelog entry
 label


      Jan 12, 2022

    

  








      

  
      



  

  @rwgk





  


    

      

  

    

        
    
    

  


      
          
  
      
            Copy link

  

      
    

  


  

      

  
    Collaborator


      

  Author


  


  

    

      

      
            rwgk
  

      

      

      commented


        Jan 12, 2022


      
    


  





      


        


  
    

      
          
Merged (as usual) via:


git checkout smart_holder
git pull . sh_merge_master
git push


      		
    

  





        


            

            
            

              
            

        

      


      

            
  

    
  
    
    

  

  


    












      

  
            

    

      
    

    


      

          @rwgk
rwgk




          removed
  the 

  needs changelog

  Possibly needs a changelog entry
 label


      Jan 12, 2022

    

  



  

  

    
  


  

      

    @rwgk
rwgk



    mentioned this pull request
    
      Feb 10, 2023
    





  


  

      
  
  Closed

  







  













  
  

    

      

  




      

    


    

      

              

  
    Sign up for free
    to join this conversation on GitHub.
    Already have an account?
    Sign in to comment


  



      

    

  




  
          




      

  

    
    

    

  

    Reviewers
  


    

    No reviews






    

  


      
  

    Assignees
  



        

    No one assigned







      


  


  

    Labels
  



    

    None yet







      

  

    

        

    Projects
  


        




    None yet




  



      


  

    
  

    Milestone
  


      No milestone





    
      
          


  

    

      
        

          
    

    Development
  



            


Successfully merging this pull request may close these issues.



  


      
    

  




      

    

  
      

  

    

      3 participants
    

    

        
          @rwgk 
        
          @Skylion007 
        
          @iwanders