Make this project compatible for drupal 9.
suggestions:

  • use drupal-check, rector , upgrade_status module to check the changes need to be done
  • Add core_version_requirement: ^8 || ^9 in thermometer.info.yml file.
CommentFileSizeAuthor
#2 d9_compatibility-3135811-2.patch1.01 KBsarvjeetsingh

Comments

sarvjeetsingh created an issue. See original summary.

sarvjeetsingh’s picture

sarvjeetsingh commented
Assigned: sarvjeetsingh » Unassigned
Status: Active » Needs review
StatusFileSize
new d9_compatibility-3135811-2.patch1.01 KB

Fixed the errors that were found using drupal-check , rector and upgrade_status module.
Added core_version_requirement: ^8 || ^9 in thermometer.info.yml file.

meghasharma’s picture

meghasharma commented
Status: Needs review » Reviewed & tested by the community

Have run drupal-check & rector tools and scanned upgrade_status on hover_css module: No errors reported.
it looks good to me..

my_site_name_dir % vendor/bin/drupal-check -ad web/modules/contrib/thermometer

 2/2 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%


                                                                                                                        
 [OK] No errors                                                                                                         
                                                                                                                        

my_site_name_dir % vendor/bin/rector process web/modules/contrib/thermometer --dry-run
Rector 0.8.x-dev@8896f0a
Config file: rector.yml

 6/6 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

                                                                                                                        
 [OK] Rector is done!
meghasharma’s picture

meghasharma commented

  • cwells committed 05e8d52 on 8.x-1.x 
    Issue #3135811 by sarvjeetsingh,meghasharma,cwells: Drupal 9 readiness
chrisfromredfin’s picture

chrisfromredfin
he/him/his
Primary language English
Location Portland, Maine
commented
Status: Reviewed & tested by the community » Fixed

Pushed to 8.x-1.x branch. Thanks!

I will be making a new 1.x branch shortly (soon as I figure out the process... :) )

Status: Fixed » Closed (fixed)

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