I set the issue status to Needs work as the following drupal-check -ad issues / errors:

drupal-check -ad web/modules/config_inspector
 9/9 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

 ------ -----------------------------------------------------------------
  Line   config_inspector.drush.inc                                                                                             
 ------ -----------------------------------------------------------------
  11     Function config_inspector_drush_command not found while trying to analyse it - autoloading is probably not configured properly.                                                                                                              
  34     Function drush_config_inspector_inspect_config not found while trying to analyse it - autoloading is probably not configured properly.                                                                                                   
  37     Call to deprecated function drush_get_option().                                                                        
  38     Call to deprecated function drush_get_option().                                                                        
  39     Call to deprecated function drush_get_option().                                                                        
 ------ -----------------------------------------------------------------

 ------ -----------------------------------------------------------------
  Line   src/ConfigInspectorManager.php                                               
 ------ -----------------------------------------------------------------
  37     Plugin manager has cache backend specified but does not declare cache tags.  
 ------ -----------------------------------------------------------------
                                                                                              
 [ERROR] Found 6 errors

Comments

ilchovuchkov created an issue. See original summary.

vuil’s picture

Issue tags: +D9Readiness
vuil’s picture

Status: Active » Needs review
vuil’s picture

Issue summary: View changes
Status: Needs review » Needs work
vuil’s picture

Status: Needs work » Active

Separate the issue into children issues, set it to "Active" until all (separated) children issues are fixed!

vuil’s picture

Status: Active » Needs review

$ drupal-check web/modules/config_inspector

 8/8 [============================] 100%

 [OK] No errors
vuil’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

vuil’s picture

Kristen Pol’s picture

Issue tags: -D9Readiness +Drupal 9 compatibility

Per a Slack discussion with Gábor Hojtsy regarding usage of D9 tags (Drupal 9, Drupal 9 compatibility, Drupal 9 readiness, etc.), "Drupal 9 compatibility" should be used for contributed projects that need updating and "Drupal 9" was the old tag for D8 issues before the D9 branch was ready. Doing tag cleanup here based on that discussion.

vuil’s picture

Status: Closed (fixed) » Needs review

I set the issue back to Needs review, because I added a child issue with the Drupal 9 compliant patch for tests (D9.x).

vuil’s picture

Status: Needs review » Needs work

I set the issue status to Needs work as the following drupal-check -ad issues:

drupal-check -ad web/modules/config_inspector
 9/9 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

 ------ -----------------------------------------------------------------
  Line   config_inspector.drush.inc                                                                                             
 ------ -----------------------------------------------------------------
  11     Function config_inspector_drush_command not found while trying to analyse it - autoloading is probably not configured properly.                                                                                                              
  34     Function drush_config_inspector_inspect_config not found while trying to analyse it - autoloading is probably not configured properly.                                                                                                   
  37     Call to deprecated function drush_get_option().                                                                        
  38     Call to deprecated function drush_get_option().                                                                        
  39     Call to deprecated function drush_get_option().                                                                        
 ------ -----------------------------------------------------------------

 ------ -----------------------------------------------------------------
  Line   src/ConfigInspectorManager.php                                               
 ------ -----------------------------------------------------------------
  37     Plugin manager has cache backend specified but does not declare cache tags.  
 ------ -----------------------------------------------------------------
                                                                                              
 [ERROR] Found 6 errors
vuil’s picture

Issue summary: View changes

I updated the issue summary with the latest error for D9.x compliance which need to be fixed.

vuil’s picture

Status: Needs work » Fixed
Related issues: +#3047418: Drush 10 command support

The drush commands issue are in another issue #3047418: Drush 10 command support, so I'll close the issue as Fixed, and add the related issue.

The issue is fixed because the Drupal module code is "up-to-date". The drush commands are a separate functionality.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.