- Issue created by @jayprakash.kushwah
- ๐ฎ๐ณIndia mithun s Bangalore
Mithun S โ made their first commit to this issueโs fork.
- Status changed to Needs review
10 months ago 8:34am 1 March 2024 - ๐ฎ๐ณIndia mithun s Bangalore
Fixed the Phpcs issues for the module and raised an MR to the branch 1.0.x
Changing the status of issue to Needs review. - Status changed to Needs work
10 months ago 6:26am 4 March 2024 - ๐ต๐ญPhilippines paraderojether
Hi Mithun S,
I reviewed MR!3, the MR is not applied cleanly and there are still remaining warnings shown below:
jetherparadero@PrometSources-MacBook-Pro custom_css_class_to_body % curl https://git.drupalcode.org/project/custom_css_class_to_body/-/merge_requests/3.diff | patch -p1 % Total % Received % Xferd Average Speed Time Time Time Current Dload Upload Total Spent Left Speed 100 4309 0 4309 0 0 1701 0 --:--:-- 0:00:02 --:--:-- 1705 patching file README.md patching file custom_css_class_to_body.info.yml 1 out of 1 hunks failed--saving rejects to custom_css_class_to_body.info.yml.rej patching file custom_css_class_to_body.module jetherparadero@PrometSources-MacBook-Pro custom_css_class_to_body % cd .. jetherparadero@PrometSources-MacBook-Pro contrib % phpcs --standard="Drupal,DrupalPractice" --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml custom_css_class_to_body FILE: /Users/jetherparadero/d_o/drupal-issues/web/modules/contrib/custom_css_class_to_body/custom_css_class_to_body.info.yml ---------------------------------------------------------------------------------------------------------------------------- FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 1 LINE ---------------------------------------------------------------------------------------------------------------------------- 1 | WARNING | Remove "project" from the info file, it will be added by drupal.org packaging automatically 1 | WARNING | Remove "datestamp" from the info file, it will be added by drupal.org packaging automatically 1 | WARNING | Remove "version" from the info file, it will be added by drupal.org packaging automatically ---------------------------------------------------------------------------------------------------------------------------- Time: 106ms; Memory: 10MB
Kindly check and advise.
Thank you. - ๐ฎ๐ณIndia dev2.addweb
nilesh.addweb โ made their first commit to this issueโs fork.
- Status changed to Needs review
5 months ago 11:31am 22 July 2024 - ๐ฎ๐ณIndia dev2.addweb
Hi,
I have resolved conflict from upstream and updated MR. Please review it. - Status changed to RTBC
5 months ago 1:12pm 22 July 2024 - ๐ฎ๐ณIndia riddhi.addweb
I have applied the MR !3 and it resolves the errors.
Please check the screenshot for the same.