Change OrderedSet.__repr__ to use OrderedSet.of

I believe OrderedSet.of(1, 2, 3) is slightly more readable than OrderedSet([1, 2, 3]).
It avoids an extra pair of brackets.

Author: Techcable

README.md

@@ -12,7 +12,7 @@ A simple and efficient pure-python ordered set.
 from techcable.orderedset import OrderedSet
 
 # prints {1, 2, 7, 3}
-print(OrderedSet([1, 2, 7, 2, 3]))
+print(OrderedSet.of(1, 2, 7, 2, 3))
 ```
 
 Supports [pydantic](pydantic.org) validation & serialization:
@@ -21,9 +21,9 @@ import pydantic
 from techcable.orderedset import OrderedSet
 
 model = pydantic.TypeAdapter(OrderedSet[int])
-# prints OrderedSet([1,2,7,8])
+# prints OrderedSet.of(1,2,7,8)
 print(repr(model.validate_python([1,2,7,8])))
-assert model.dump_python(OrderedSet([1,2,7,8])) == [1,2,7,8]
+assert model.dump_python(OrderedSet.of(1,2,7,8)) == [1,2,7,8]
 ```
 
 ## Potential Future Features

src/techcable/orderedset/_orderedset.py

@@ -241,7 +241,8 @@ def copy(self) -> OrderedSet[T]:
     @override
     def __repr__(self) -> str:
         # by convention, this should roundtrip through eval
-        return f"OrderedSet([{', '.join(map(repr, self))}])"
+        # in v0.1.6, this changed to use OrderedSet.of(1,2,3) instead of OrderedSet([1,2,3])
+        return f"OrderedSet.of({', '.join(map(repr, self))})"
 
     @override
     def __str__(self) -> str:

tests/test_simple.py

@@ -58,8 +58,9 @@ def test_dedup():
 
 def test_format():
     # OrderedSet.__repr__ needs to round trip through eval
-    assert repr(OrderedSet([1, 2, 3])) == "OrderedSet([1, 2, 3])"
-    assert repr(OrderedSet(["foo"])) == "OrderedSet(['foo'])"
+    # since v0.1.6 we use of OrderedSet.of
+    assert repr(OrderedSet([1, 2, 3])) == "OrderedSet.of(1, 2, 3)"
+    assert repr(OrderedSet(["foo"])) == "OrderedSet.of('foo')"
     # OrderedSet.__str__ has no such requirement
     assert str(OrderedSet([1, 2, 3])) == "{1, 2, 3}"
     # however, it still should call repr on each element, not str