Apply suggestions from code review

Co-authored-by: Adam Wathan <adam.wathan@gmail.com>

Author: philipp-spiess

CHANGELOG.md

@@ -18,7 +18,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - _Upgrade (experimental)_: Minify arbitrary values when printing candidates ([#14720](https://github.com/tailwindlabs/tailwindcss/pull/14720))
 - _Upgrade (experimental)_: Ensure legacy theme values ending in `1` (like `theme(spacing.1)`) are correctly migrated to custom properties ([#14724](https://github.com/tailwindlabs/tailwindcss/pull/14724))
 - _Upgrade (experimental)_: Migrate arbitrary values to bare values for the `from-*`, `via-*`, and `to-*` utilities ([#14725](https://github.com/tailwindlabs/tailwindcss/pull/14725))
-- _Upgrade (experimental)_: Reduce the number of false-positive migration of the important modifier (e.g. `!border` to `border!`) ([#14737](https://github.com/tailwindlabs/tailwindcss/pull/14737))
+- _Upgrade (experimental)_: Don't migrate important modifiers that are actually logical negations (e.g. `let foo = !border` to `let foo = border!`) ([#14737](https://github.com/tailwindlabs/tailwindcss/pull/14737))
 
 ### Changed
 

packages/@tailwindcss-upgrade/src/template/codemods/important.ts

@@ -30,7 +30,7 @@ export function important(
       // The important migration is one of the most broad migrations with a high
       // potential of matching false positives since `!` is a valid character in
       // most programming languages. Since v4 is technically backward compatible
-      // to v3 in that it can read `!` in the front of the utility too, we err
+      // with v3 in that it can read `!` in the front of the utility too, we err
       // on the side of caution and only migrate candidates that we are certain
       // are inside of a string.
       if (location) {