Apply suggestions from code review

Co-authored-by: Ivan Dlugos <6349682+vaind@users.noreply.github.com>

Author: lucas-zimerman

danger/README.md

@@ -37,8 +37,8 @@ jobs:
 
 * `extra-install-packages`: Additional packages that are required by the extra-dangerfile, you can find a list of packages here: https://packages.debian.org/search?suite=bookworm&keywords=curl.
   * type: string
-  required: false
-  default: ""
+  * required: false
+  * default: ""
 
 ## Outputs
 

danger/action.yml

@@ -80,9 +80,8 @@ runs:
       env:
         EXTRA_INSTALL_PACKAGES: ${{ inputs.extra-install-packages }}
       run: |
-        docker exec --user root danger apt-get update
         echo "Installing packages: $EXTRA_INSTALL_PACKAGES"
-        docker exec --user root danger sh -c "set -e && apt-get install -y --no-install-recommends $EXTRA_INSTALL_PACKAGES"
+        docker exec --user root danger sh -c "set -e && apt-get update && apt-get install -y --no-install-recommends $EXTRA_INSTALL_PACKAGES"
         echo "All additional packages installed successfully."
 
     - name: Run DangerJS

danger/dangerfile.js

@@ -186,7 +186,7 @@ async function checkActionsArePinned() {
   }
 }
 
-async function CheckFromExternalChecks() {
+async function checkFromExternalChecks() {
   // Get the external dangerfile path from environment variable (passed via workflow input)
   // Priority: EXTRA_DANGERFILE (absolute path) -> EXTRA_DANGERFILE_INPUT (relative path)
   const extraDangerFilePath = process.env.EXTRA_DANGERFILE || process.env.EXTRA_DANGERFILE_INPUT;
@@ -227,7 +227,7 @@ async function checkAll() {
   await checkDocs();
   await checkChangelog();
   await checkActionsArePinned();
-  await CheckFromExternalChecks();
+  await checkFromExternalChecks();
 }
 
 schedule(checkAll);