search

ValeevGroup / SeQuant

SeQuant: Symbolic Algebra of Tensors over Operators and Scalars
GNU General Public License v3.0
17 stars 6 forks source link

CI: Add formatting check #194

Closed Krzmbrzl closed 6 months ago

Krzmbrzl commented 6 months ago

@evaleev once this is merged, all open PRs should be rebased/merged with master to ensure the check applies to them as well (to avoid pulling in changes that undo the formatting streamlining again).

ajay-mk commented 6 months ago

Should we also exclude catch.hpp from this?

Krzmbrzl commented 6 months ago

We certainly could - though once it is formatted it shouldn't be a concern, no?

ajay-mk commented 6 months ago

Oh, I didn't see that you already formatted it.

evaleev commented 6 months ago

I undid the catch.hpp format, will remove embedded file and will use FC to catch2 (via https://github.com/ValeevGroup/SeQuant/pull/197 )