trap "echo \"Finished on $(date)\"" EXIT
trap 'echo "Finished on $(date)"' EXIT
With double quotes, all parameter and command expansions will expand when the trap is defined rather than when it's executed.
In the example with the Problematic code, the message will contain the date on which the trap was declared, and not the date on which the script exits.
Using single quotes will prevent expansion at declaration time, and save it for execution time.
If you don't care that the trap code is expanded early because the commands/variables won't change during execution of the script, or because you want to use the current and not the future values, then you can ignore this message.
ShellCheck is a static analysis tool for shell scripts. This page is part of its documentation.