Skip to content
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

Arc-ify IoOwn #3663

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Arc-ify IoOwn #3663

wants to merge 1 commit into from

Conversation

mikea
Copy link
Contributor

@mikea mikea commented Mar 7, 2025

All const method changes were (correctly) forced upon me by compiler

This demonstrates (and depends on) capnproto/capnproto#2251

kentonv
kentonv reviewed Mar 7, 2025
View reviewed changes
@mikea
Arc-ify IoOwn
f71c144 
All const method changes were (correctly) forced upon me by compiler
@mikea mikea force-pushed the maizatskyi/2025-03-07-io-ptr-arc branch from a23daa0 to f71c144 Compare March 7, 2025 17:03
@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 7, 2025

The generated output of @cloudflare/workers-types has been changed by this PR. If this is intentional, run bazel build //types && rm -rf types/generated-snapshot && cp -r bazel-bin/types/definitions types/generated-snapshot to update the snapshot. Alternatively, you can download the full generated types:

Full Type Diff 




      		
    

  





        


            

            

              
            

        

      


      

            
  

    
  
    
    

  

  



    
















  
  

    

      

  




    


    

      

              

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


  



      

    

  




  
          




      

  

    
    

    
  

    Reviewers
  


    


        

          

  
      @kentonv
  
    kentonv



          
            
              
            
          
          kentonv left review comments

        


        

      At least 1 approving review is required to merge this pull request.
  








    

  


      
  

    Assignees
  



        

    No one assigned







      


  


  

    Labels
  



    

    None yet







      

  

    

        

    Projects
  


        




    None yet




  



      


  

    
  

    Milestone
  


      No milestone





    
      
          


  

    

      
        

          
    

    Development
  



            


Successfully merging this pull request may close these issues.



  


      
    

  




      

    

  
      

  

    

      2 participants
    

    

        
          @mikea 
        
          @kentonv