{ "uuid": [ { "value": "cb064b44-6a73-4e6e-88d6-fd970cd3bcc0" } ], "langcode": [ { "value": "en" } ], "type": [ { "target_id": "daily_email", "target_type": "node_type", "target_uuid": "8bde1f2f-eef9-4f2d-ae9c-96921f8193d7" } ], "revision_timestamp": [ { "value": "2025-04-16T14:12:57+00:00" } ], "revision_uid": [ { "target_type": "user", "target_uuid": "b8966985-d4b2-42a7-a319-2e94ccfbb849" } ], "revision_log": [], "status": [ { "value": true } ], "uid": [ { "target_type": "user", "target_uuid": "b8966985-d4b2-42a7-a319-2e94ccfbb849" } ], "title": [ { "value": "Code reviews are about the code, not code style" } ], "created": [ { "value": "2024-10-30T00:00:00+00:00" } ], "changed": [ { "value": "2025-04-16T14:12:57+00:00" } ], "promote": [ { "value": false } ], "sticky": [ { "value": false } ], "default_langcode": [ { "value": true } ], "revision_translation_affected": [ { "value": true } ], "path": [ { "alias": "\/daily\/2024\/10\/30\/code-reviews-are-about-the-code-not-code-style", "langcode": "en" } ], "body": [ { "value": "\n
If you do code reviews, they should be about reviewing the code and not about the style of the code.<\/p>\n\n
There shouldn't be comments about whether tabs or spaces are used, how many spaces are on each line, where the braces are, or whether there should be semicolons.<\/p>\n\n
A code style should be defined upfront and checking code against it can be done automatically with tools such as phpcs or Prettier.<\/p>\n\n
You don't need to wait for a human to review the code style - automated tools can be run locally or in a CI pipeline and provide feedback much faster.<\/p>\n\n
And if people aren't reviewing the code style, they can focus on reviewing the code itself.<\/p>\n\n ", "format": "full_html", "processed": "\n
If you do code reviews, they should be about reviewing the code and not about the style of the code.<\/p>\n\n
There shouldn't be comments about whether tabs or spaces are used, how many spaces are on each line, where the braces are, or whether there should be semicolons.<\/p>\n\n
A code style should be defined upfront and checking code against it can be done automatically with tools such as phpcs or Prettier.<\/p>\n\n
You don't need to wait for a human to review the code style - automated tools can be run locally or in a CI pipeline and provide feedback much faster.<\/p>\n\n
And if people aren't reviewing the code style, they can focus on reviewing the code itself.<\/p>\n\n ", "summary": null } ], "feeds_item": [ { "imported": "2025-04-16T14:12:57+00:00", "guid": null, "hash": "9fe14e0414ae452819b91e9a2abc8403", "target_type": "feeds_feed", "target_uuid": "90c85284-7ca8-4074-9178-97ff8384fe76" } ] }