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

GitHub Actions Code Formatter workflow #5221

Merged
merged 1 commit into from
Mar 4, 2024
Merged

GitHub Actions Code Formatter workflow #5221

merged 1 commit into from
Mar 4, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Mar 4, 2024

This pull request includes updates from the GitHub Actions Code Formatter workflow. Please review the changes and merge if everything looks good.

@github-actions github-actions bot added the code quality For PRs that improve, reformat, or refactor code label Mar 4, 2024
@github-actions github-actions bot requested review from a team as code owners March 4, 2024 05:00
@ASTRobinson ASTRobinson mentioned this pull request Mar 4, 2024
Closed
@ASTRobinson ASTRobinson closed this Mar 4, 2024
@ASTRobinson ASTRobinson deleted the date_2024_03_04 branch March 4, 2024 11:47
@ASTRobinson ASTRobinson restored the date_2024_03_04 branch March 4, 2024 11:48
@ASTRobinson ASTRobinson reopened this Mar 4, 2024
@github-actions github-actions bot added the environments-repository Used to exclude PRs from this repo in our Slack PR update label Mar 4, 2024
@modernisation-platform-ci
Copy link
Contributor

@ASTRobinson Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@ASTRobinson ASTRobinson had a problem deploying to electronic-monitoring-data-development March 4, 2024 11:49 — with GitHub Actions Error
@ASTRobinson ASTRobinson had a problem deploying to hmpps-oem-development March 4, 2024 11:49 — with GitHub Actions Error
@ASTRobinson ASTRobinson had a problem deploying to corporate-staff-rostering-development March 4, 2024 11:49 — with GitHub Actions Error
@ASTRobinson ASTRobinson had a problem deploying to digital-prison-reporting-development March 4, 2024 11:49 — with GitHub Actions Error
@ASTRobinson ASTRobinson had a problem deploying to hmpps-domain-services-test March 4, 2024 11:49 — with GitHub Actions Error
@ASTRobinson ASTRobinson had a problem deploying to hmpps-domain-services-development March 4, 2024 11:50 — with GitHub Actions Error
@github-actions GitHub Actions
Copy link
Contributor Author

github-actions bot commented Mar 4, 2024

TFSEC Scan Success

Show Output ```hcl

TFSEC will check the following folders:

</details> #### `Checkov Scan` Success
<details><summary>Show Output</summary>

```hcl

*****************************

Checkov will check the following folders:

CTFLint Scan Success

Show Output 
*****************************

Setting default tflint config...
Running tflint --init...
Installing "terraform" plugin...
Installed "terraform" (source: github.com/terraform-linters/tflint-ruleset-terraform, version: 0.5.0)
tflint will check the following folders:

Trivy Scan

Show Output 



      		
    

  





        


            

            

              
            

        

      


      

            
  

    
  
    
    

  

  



    












        

  
          

  

  
  

  
      @ASTRobinson
  ASTRobinson


            had a problem deploying
      to
        digital-prison-reporting-test



    March 4, 2024 11:50      — with 
GitHub Actions

                
    Error


    





  

  


  

  


  

  







        

  
        

  

      

  ASTRobinson
  

  
          

            
                  ASTRobinson
  

            

            approved these changes


            
                
                  Mar 4, 2024
                
            
            
          


          



              
    View reviewed changes
  


          





          

  
  
  
              

                        

  

    

        
    
    

  


      
          
  
      
            Copy link

  

      
    

  


  

      

  
    Contributor


      

  


  

    

      @ASTRobinson

      
            ASTRobinson
  

      

      

      left a comment




      
    


  



                    

                        

  

  
 

  
    

    
Choose a reason for hiding this comment


    

      The reason will be displayed to describe this comment to others. Learn more.
    


    

      
      



  



  
    

      
LGTM - linting changes only.

    

  
  




                      

                    
  

    
  
    
    

  

  




                        

                      
                    








      

  
  
  




  








        

  
          

  

    
  

  


        
        @ASTRobinson
  ASTRobinson



        merged commit ecd21f8
      into

      
  
    main
  


    Mar 4, 2024

      

        111 of 130 checks passed
      


      

        
          
  
    
    

        
      

  







  

  
  

  
      @ASTRobinson
  ASTRobinson


    
    deleted the
    
      
        date_2024_03_04
    
    branch

    March 4, 2024 11:52    
    











  
  

    

      

  




    


    

      

              

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


  



      

    

  




  
          




      

  

    
    

    
  

    Reviewers
  


    


        

          

  
      @ASTRobinson
  
    ASTRobinson



          
            
              
            
          
          ASTRobinson approved these changes

        


      






    

  


      
  

    Assignees
  



        

    No one assigned







      


  


  

    Labels
  



    

      

    code quality

  For PRs that improve, reformat, or refactor code
  

    environments-repository

  Used to exclude PRs from this repo in our Slack PR update








      

  

    

        

    Projects
  


        




    None yet




  



      


  

    
  

    Milestone
  


      No milestone





    
      
          


  

    
      

        
    

    Development
  



          


Successfully merging this pull request may close these issues.



  


    
  




      

    

  
      

  

    

      2 participants
    

    

        
          @modernisation-platform-ci 
        
          @ASTRobinson