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

modify appcontainers readme.typescript.md #20059

Merged
merged 1 commit into from
Aug 2, 2022
Merged

modify appcontainers readme.typescript.md #20059

merged 1 commit into from
Aug 2, 2022

Conversation

kazrael2119
Copy link
Contributor

@openapi-workflow-bot
Copy link

Hi, @kazrael2119 Thanks for your PR. I am workflow bot for review process. Here are some small tips.

  • Please ensure to do self-check against checklists in first PR comment.
  • PR assignee is the person auto-assigned and responsible for your current PR reviewing and merging.
  • For specs comparison cross API versions, Use API Specs Comparison Report Generator
  • If there is CI failure(s), to fix CI error(s) is mandatory for PR merging; or you need to provide justification in PR comment for explanation. How to fix?

    • Any feedback about review process or workflow bot, pls contact swagger and tools team. vscswagger@microsoft.com

    @openapi-pipeline-app
    Copy link

    openapi-pipeline-app bot commented Aug 2, 2022 •
    edited
    Loading

    Swagger Validation Report

    ️️✔️BreakingChange succeeded [Detail] [Expand] 
    There are no breaking changes.
    ️️✔️Breaking Change(Cross-Version) succeeded [Detail] [Expand] 
    There are no breaking changes.
    ️⌛CredScan pending [Detail]
    ️️✔️LintDiff succeeded [Detail] [Expand] 
    Validation passes for LintDiff.
    compared tags (via openapi-validator v1.13.0) new version base version
    default default(703cc47) default(main)
    ️️✔️Avocado succeeded [Detail] [Expand] 
    Validation passes for Avocado.
    ️️✔️ApiReadinessCheck succeeded [Detail] [Expand]
    ️️✔️ModelValidation succeeded [Detail] [Expand] 
    Validation passes for ModelValidation.
    ️️✔️SemanticValidation succeeded [Detail] [Expand] 
    Validation passes for SemanticValidation.
    ️⌛PoliCheck pending [Detail]
    ️️✔️SDK Track2 Validation succeeded [Detail] [Expand] 
    Validation passes for SDKTrack2Validation
    ️️✔️PrettierCheck succeeded [Detail] [Expand] 
    Validation passes for PrettierCheck.
    ️️✔️SpellCheck succeeded [Detail] [Expand] 
    Validation passes for SpellCheck.
    ️️✔️Lint(RPaaS) succeeded [Detail] [Expand] 
    Validation passes for Lint(RPaaS).
    ️️✔️CadlValidation succeeded [Detail] [Expand] 
    Validation passes for CadlValidation.
    Posted by Swagger Pipeline | How to fix these errors?

    @openapi-pipeline-app
    Copy link

    openapi-pipeline-app bot commented Aug 2, 2022 •
    edited
    Loading

    Swagger Generation Artifacts

    ️️✔️ApiDocPreview succeeded [Detail] [Expand]
    ️️✔️SDK Breaking Change Tracking succeeded [Detail] [Expand]

    Breaking Changes Tracking

    ️✔️azure-sdk-for-go - armappcontainers - 2.0.0
    ️✔️azure-sdk-for-python-track2 - track2_azure-mgmt-appcontainers - 1.0.0
    ️⚠️ azure-sdk-for-python-track2 warning [Detail]   
  
    • ⚠️Warning [Logs]Release - Generate from d94ec99. SDK Automation 14.0.0 
      command	sh scripts/automation_init.sh ../azure-sdk-for-python_tmp/initInput.json ../azure-sdk-for-python_tmp/initOutput.json
cmderr	[automation_init.sh] WARNING: Skipping azure-nspkg as it is not installed.
command	sh scripts/automation_generate.sh ../azure-sdk-for-python_tmp/generateInput.json ../azure-sdk-for-python_tmp/generateOutput.json
cmderr	[automation_generate.sh] WARN config global `--global`, `--local` are deprecated. Use `--location=global` instead.
cmderr	[automation_generate.sh]
cmderr	[automation_generate.sh] npm notice New minor version of npm available! 8.11.0 -> 8.15.1
cmderr	[automation_generate.sh] npm notice Changelog: <https://github.com/npm/cli/releases/tag/v8.15.1>
cmderr	[automation_generate.sh] npm notice Run `npm install -g npm@8.15.1` to update!
cmderr	[automation_generate.sh] npm notice
    • ️✔️track2_azure-mgmt-appcontainers        [View full logs]         [Release SDK Changes]        Breaking Change Detected
      info	[Changelog] **Breaking changes**
info	[Changelog]
info	[Changelog]   - Model CustomHostnameAnalysisResult no longer has parameter id
info	[Changelog]   - Model CustomHostnameAnalysisResult no longer has parameter name
info	[Changelog]   - Model CustomHostnameAnalysisResult no longer has parameter system_data
info	[Changelog]   - Model CustomHostnameAnalysisResult no longer has parameter type
    ️️✔️ azure-sdk-for-java succeeded [Detail] [Expand]  
  
    ️️✔️ azure-sdk-for-go succeeded [Detail] [Expand]  
  
    • ️✔️Succeeded [Logs]Release - Generate from d94ec99. SDK Automation 14.0.0 
      command	sh ./eng/scripts/automation_init.sh ../../../../../azure-sdk-for-go_tmp/initInput.json ../../../../../azure-sdk-for-go_tmp/initOutput.json
command	generator automation-v2 ../../../../../azure-sdk-for-go_tmp/generateInput.json ../../../../../azure-sdk-for-go_tmp/generateOutput.json
    • ️✔️armappcontainers        [View full logs]         [Release SDK Changes]        Breaking Change Detected
      info	[Changelog] ### Breaking Changes
info	[Changelog]
info	[Changelog] - Function `*CertificatesClient.CreateOrUpdate` parameter(s) have been changed from `(context.Context, string, string, string, *CertificatesClientCreateOrUpdateOptions)` to `(context.Context, string, string, string, Certificate, *CertificatesClientCreateOrUpdateOptions)`
info	[Changelog] - Struct `CustomHostnameAnalysisResultProperties` has been removed
info	[Changelog] - Field `CertificateEnvelope` of struct `CertificatesClientCreateOrUpdateOptions` has been removed
info	[Changelog] - Field `Name` of struct `CustomHostnameAnalysisResult` has been removed
info	[Changelog] - Field `SystemData` of struct `CustomHostnameAnalysisResult` has been removed
info	[Changelog] - Field `Type` of struct `CustomHostnameAnalysisResult` has been removed
info	[Changelog] - Field `Properties` of struct `CustomHostnameAnalysisResult` has been removed
info	[Changelog] - Field `ID` of struct `CustomHostnameAnalysisResult` has been removed
info	[Changelog]
info	[Changelog] ### Features Added
info	[Changelog]
info	[Changelog] - New field `ARecords` in struct `CustomHostnameAnalysisResult`
info	[Changelog] - New field `HasConflictOnManagedEnvironment` in struct `CustomHostnameAnalysisResult`
info	[Changelog] - New field `AlternateCNameRecords` in struct `CustomHostnameAnalysisResult`
info	[Changelog] - New field `AlternateTxtRecords` in struct `CustomHostnameAnalysisResult`
info	[Changelog] - New field `CNameRecords` in struct `CustomHostnameAnalysisResult`
info	[Changelog] - New field `TxtRecords` in struct `CustomHostnameAnalysisResult`
info	[Changelog] - New field `CustomDomainVerificationTest` in struct `CustomHostnameAnalysisResult`
info	[Changelog] - New field `ConflictingContainerAppResourceID` in struct `CustomHostnameAnalysisResult`
info	[Changelog] - New field `CustomDomainVerificationFailureInfo` in struct `CustomHostnameAnalysisResult`
info	[Changelog] - New field `HostName` in struct `CustomHostnameAnalysisResult`
info	[Changelog] - New field `IsHostnameAlreadyVerified` in struct `CustomHostnameAnalysisResult`
info	[Changelog]
info	[Changelog] Total 5 breaking change(s), 1 additive change(s).
    ️️✔️ azure-sdk-for-js succeeded [Detail] [Expand]  
  
    • ️✔️Succeeded [Logs]Release - Generate from d94ec99. SDK Automation 14.0.0 
      command	sh .scripts/automation_init.sh ../azure-sdk-for-js_tmp/initInput.json ../azure-sdk-for-js_tmp/initOutput.json
warn	File azure-sdk-for-js_tmp/initOutput.json not found to read
command	sh .scripts/automation_generate.sh ../azure-sdk-for-js_tmp/generateInput.json ../azure-sdk-for-js_tmp/generateOutput.json
    • ️✔️@azure/arm-appcontainers        [View full logs]         [Release SDK Changes]      
      info	[Changelog] **Features**
info	[Changelog]
info	[Changelog]   - Added Interface AuthConfig
info	[Changelog]   - Added Interface Certificate
info	[Changelog]   - Added Interface ContainerApp
info	[Changelog]   - Added Interface CustomHostnameAnalysisResult
info	[Changelog]   - Added Interface DaprComponent
info	[Changelog]   - Added Interface ManagedEnvironment
info	[Changelog]   - Added Interface ManagedEnvironmentStorage
info	[Changelog]   - Added Interface ProxyResource
info	[Changelog]   - Added Interface Replica
info	[Changelog]   - Added Interface Revision
info	[Changelog]   - Added Interface SourceControl
info	[Changelog]   - Added Interface TrackedResource
error	breakingChangeTracking is enabled, but version or changelogItem is not found in output.
    ️⚠️ azure-resource-manager-schemas warning [Detail]   
  
    • ⚠️Warning [Logs]Release - Generate from d94ec99. Schema Automation 14.0.0 
      command	.sdkauto/initScript.sh ../azure-resource-manager-schemas_tmp/initInput.json ../azure-resource-manager-schemas_tmp/initOutput.json
cmderr	[initScript.sh]  old lockfile
cmderr	[initScript.sh] npm WARN old lockfile The package-lock.json file was created with an old version of npm,
cmderr	[initScript.sh] npm WARN old lockfile so supplemental metadata must be fetched from the registry.
cmderr	[initScript.sh] npm WARN old lockfile
cmderr	[initScript.sh] npm WARN old lockfile This is a one-time fix-up, please be patient...
cmderr	[initScript.sh] npm WARN old lockfile
warn	File azure-resource-manager-schemas_tmp/initOutput.json not found to read
command	.sdkauto/generateScript.sh ../azure-resource-manager-schemas_tmp/generateInput.json ../azure-resource-manager-schemas_tmp/generateOutput.json
warn	No file changes detected after generation
    • ️✔️app        [View full logs]       
    ️️✔️ azure-powershell succeeded [Detail] [Expand]  
  
    • ️✔️Succeeded [Logs]Release - Generate from d94ec99. SDK Automation 14.0.0 
      command	sh ./tools/SwaggerCI/init.sh ../azure-powershell_tmp/initInput.json ../azure-powershell_tmp/initOutput.json
command	pwsh ./tools/SwaggerCI/psci.ps1 ../azure-powershell_tmp/generateInput.json ../azure-powershell_tmp/generateOutput.json
    • ️✔️Az.ContainerAppsApi        [View full logs]         [Release SDK Changes]      
    Posted by Swagger Pipeline | How to fix these errors?
          		
    
    
      

    
    


        
    

            

            
    
              
            
    
        
    
      
    

      
    
            
  
    
    
  
    
    

  
    
  
    


        
    

    


    



    


      
    
  
      

    

  
    
  @openapi-pipeline-app


    


  
    

    
    
      
    
  
    
    
    
        
    
    

  


      
          
  
      
            Copy link

  
    
              
    
    
  
    

  
    
      



      

  
    

  
    
    
    
      

      
            openapi-pipeline-app
  bot

      

      

      commented


        Aug 2, 2022


      
  •

  
    
    
      
    
        
          edited
          
        
        
      
    
        
    
      
        
      Loading

      
    
  
    
    

    
    

  
    

    


      
    

        

    
  
    
    
      
            
    Generated ApiView
    
  
    
    
    
      Language
      Package Name
      ApiView Link
    
    
      
    
        Python
        track2_azure-mgmt-appcontainers
          Create ApiView failed. Please ensure your github account in Azure/Microsoft is public and re-trigger the CI. If issue still exists, please ask PR assignee for help
      
    
      
    
        Java
        azure-resourcemanager-appcontainers
          Create ApiView failed. Please ensure your github account in Azure/Microsoft is public and re-trigger the CI. If issue still exists, please ask PR assignee for help
      
    
      
    
        JavaScript
        @azure/arm-appcontainers
          Create ApiView failed. Please ensure your github account in Azure/Microsoft is public and re-trigger the CI. If issue still exists, please ask PR assignee for help
      
    
  
    
          		
    
    
      

    
    


        
    

            

            
    
              
            
    
        
    
      
    

      
    
            
  
    
    
  
    
    

  
    
  
    


        
    

    


    



    


      
    
  
            





    

      
    
  
        



    

      
    
  
          
    
  
    
    
  
    

  
    
      
    
    @azure-sdk
azure-sdk



    mentioned this pull request
    
      Aug 2, 2022
    

    


    
  


  
    
      
   Closed

  
    

    




  
    

    

    


  
    
  
    
    
  
    

  
    
      
    
    @azure-sdk
azure-sdk



    mentioned this pull request
    
      Aug 2, 2022
    

    


    
  


  
    
      
   Closed

  
    

    




  
    

    

    


  
    
  
    
    
  
    

  
    
      
    
  This was referenced Aug 2, 2022

    


    
  


  
    
      
   Closed

  
    

    



      
    

    


    
  


  
    
      
   Closed

  
    

    



      
    

    


    
  


  
    
      
   Closed

  
    

    



  
    

    

  
    
  
    
    
  
    
  
    


        @qiaozha
  qiaozha



        merged commit d94ec99
      into

      
  
    Azure:main
  


    Aug 2, 2022

  
    

    


    

  
    
  
    
    
  
    

  
    
      
    
  This was referenced Aug 2, 2022

    


    
  


  
    
      
   Closed

  
    

    



      
    

    


    
  


  
    
      
   Closed

  
    

    



      
    

    


    
  


  
    
      
   Closed

  
    

    



      
    

    


    
  


  
    
      
   Closed

  
    

    



      
    

    


    
  


  
    
      
   Closed

  
    

    



  
    

    

  
    
  
  
    
  
          @kazrael2119
  kazrael2119


    
    deleted the
    
      
        app-readme
    
    branch

    
      August 25, 2022 02:02
    
    

    

    
  
  



    





  
  
    
    
    
      

      
    



    
    

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


  

    

      
    
    
    
  
    


    
  
              

    


      
    
  
    
    
    
    
  
    
    Reviewers
  
    

    


        
    
          

  
    @qiaozha
  
    qiaozha



          
            
              
            
          
          qiaozha approved these changes

        
    

      
    


    


    
    
  
    

      
  
    
    Assignees
  
    


        

      
    
        

  
    @leni-msft
  
    leni-msft
    

      
    
    



    


      

    
  


  
    
    Labels
  
    


    
    
      

    CI-BreakingChange-Go

  

    resource-manager



    


    


      

  
    
    
    
        
    
    Projects
  
    

        
    

    

    None yet




      
    


      

    
  
    
    
  
    
    Milestone
  
    

      No milestone


    


    
      
          

    
  
    
    
      
    
        
    
    
    Development
  
    


          


    Successfully merging this pull request may close these issues.
    


  


            
  
    

    

        
          

    

  
      
    
  
    
    
    
      3 participants
    
    
    
    
        
          @kazrael2119 
        
          @qiaozha 
        
          @leni-msft