update lying doc

also disable TestHttps because it's annoying

Author: Simn

std/sys/thread/Tls.hx

@@ -27,19 +27,17 @@ package sys.thread;
 #end
 
 /**
-	Creates thread local storage.
-
-	(hl) Warning: At the moment `Tls` does not protect the value from being
-	garbage collected. Keep the value reachable to avoid crashes.
+	A thread-local storage to store per-thread values.
 **/
 extern class Tls<T> {
 	var value(get, set):T;
 
 	/**
 		Creates thread local storage. This is placeholder that can store
 		a value that will be different depending on the local thread.
-		Set the tls value to `null` before exiting the thread
-		or the memory will never be collected.
+
+		It is good practice to set the value to `null` when the thread exits.
+		Otherwise, it might not be garbage-collected on some targets.
 	**/
 	function new():Void;
 }

tests/unit/src/unit/TestMain.hx

@@ -66,7 +66,9 @@ function main() {
 		new TestHashMap(),
 		new TestRest(),
 		#if (!php && !lua)
-		new TestHttps(),
+		/* This is annoying and causes spurious CI failures. Let's just make an effort to
+			not break it! */
+		// new TestHttps(),
 		#end
 		/* Every target struggles with these tests. We can occasionally enable them when
 			making related changes, but otherwise this is not worth the CI time. */