fix(concurrency): remove unused sync cond

[danwt]

This thing was useless, AFAIU

PR Standards

Opening a pull request should be able to meet the following requirements

---

Close #XXX

<-- Briefly describe the content of this pull request -->

For Author:

• Targeted PR against correct branch
• included the correct type prefix in the PR title
• Linked to Github issue with discussion and accepted design
• Targets only one github issue
• Wrote unit and integration tests
• All CI checks have passed
• Added relevant godoc comments

---

For Reviewer:

• confirmed the correct type prefix in the PR title
• Reviewers assigned
• confirmed all author checklist items have been addressed

---

After reviewer approval:

• In case targets main branch, PR should be squashed and merged.
• In case PR targets a release branch, PR should be rebased.

[mtsitrin]

I think the entire snippet can be removed

// Check if after we sync we are synced or a new syncTarget was already set.
// If we are synced then signal all goroutines waiting on isSyncedCond.
if m.store.Height() >= m.syncTarget.Load() {
    m.logger.Info("Synced at height", "height", m.store.Height())
    m.isSyncedCond.L.Lock()
    m.isSyncedCond.Signal()
    m.isSyncedCond.L.Unlock()
}

wdyt