workflow - fix hemtt lint, drop sqflint, fix ext clippy warnings

[PabstMirror]

(Will only apply after merge)

• Fix hemtt lint I think
  ref https://github.com/acemod/ACE3/actions/runs/10434328035/job/28897068966?pr=10144
  switch to xcopy /y use this to overwrite files

• Disable sqflint, hasn't been updated in years (it doesn't understand #),
  I don't think anyone looks at it's output?
  it does have the ability to catch some things like undefined/shadowed vars
  but it has too many false positives to be useful imho

• Fix extension lint
  ref https://github.com/acemod/ACE3/actions/runs/10434327964/job/28897069005

  thread 'main' panicked at extension/build.rs:2:46:
  Open git repo: Error { code: -36, klass: 7, message: "repository path '/__w/ACE3/ACE3' is not owned by current user" }

possibly related to libgit2/libgit2#6279
but I don't really understand as checkout is doing

Adding repository directory to the temporary git global config as a safe directory
/usr/bin/git config --global --add safe.directory /__w/ACE3/ACE3

in any case it shouldn't hurt to make it buildable without git?

[PabstMirror]

failing when building on linux
if cfg!(feature = "clipboard") doesn't seem to be working?

error[E0433]: failed to resolve: use of undeclared crate or module `clipboard`
  --> extension/src/lib.rs:26:17
   |
26 |                 clipboard::group()
   |                 ^^^^^^^^^ use of undeclared crate or module `clipboard`

[PabstMirror]

ugly but should work?

#[cfg(feature = "clipboard")]
#[arma]
fn init() -> Extension {
    Extension::build()
        .version(env!("GIT_HASH").to_string())
        .group("artillery", artillery::group())
        .group("ballistics", ballistics::group())
        .group("clipboard", clipboard::group())
        .command("break_line", break_line::break_line)
        .command("fcs", fcs::get_solution)
        .command("version", || env!("GIT_HASH"))
        .finish()
}

#[cfg(not(feature = "clipboard"))]
#[arma]
fn init() -> Extension {
    Extension::build()
        .version(env!("GIT_HASH").to_string())
        .group("artillery", artillery::group())
        .group("ballistics", ballistics::group())
        .command("break_line", break_line::break_line)
        .command("fcs", fcs::get_solution)
        .command("version", || env!("GIT_HASH"))
        .finish()
}

[BrettMayson]

I can't remember why I added that feature, potentially it's not needed?

[PabstMirror]

hemtt check fails, but should work after merge

[BrettMayson]

Removed the #[must_use]?

[BrettMayson]

Personally I don't like commenting out things, if we don't need it let's remove it, we'll likely not need this again and if we do we can find this PR