refactor(ci): Minimize skip list - test more features

Only skip features that truly can't be tested:
- Platform-specific (x11, bitcoinconsensus)
- Broken (dashcore no-std)
- Dev flags

Now testing: bincode, serde, schemars, bls, eddsa, etc.
Config reduced from 83 to 38 lines.

Author: xdustinface

.github/ci-features.yml

@@ -1,83 +1,37 @@
 # Feature testing configuration
 #
 # CI auto-detects features from Cargo.toml and tests ALL of them except:
-# - 'skip': features to skip (with reason)
+# - 'skip': features that CAN'T be tested (with reason)
 # - 'no-std': explicit no-std build checks (optional)
 #
 # If a crate isn't listed here, all its features are tested.
 
 dashcore_hashes:
   skip:
-    - default           # Tested via --all-features in main test job
-    - x11               # Platform-specific, requires rs-x11-hash native lib
+    - x11               # Requires native rs-x11-hash lib (platform-specific)
     - rs-x11-hash       # Internal feature for x11
-    - bincode           # Internal serialization
-    - core2             # Compatibility shim
-    - schemars          # JSON schema generation, optional tooling
-    - actual-schemars   # Internal feature for schemars
   no-std:
-    - bare              # --no-default-features
-    - alloc             # --no-default-features --features alloc
-    - alloc serde       # --no-default-features --features "alloc serde"
+    - bare
+    - alloc
+    - alloc serde
 
 dashcore:
   skip:
-    - default           # Tested via --all-features in main test job
-    - no-std            # Broken, needs investigation
-    - bincode           # Internal serialization
-    - base64-compat     # Compatibility wrapper
-    - bitcoinconsensus  # Requires native lib, not commonly used
-    - blsful            # Internal BLS implementation detail
-    - core2             # Compatibility shim
-    - core-block-hash-use-x11  # Platform-specific x11 dependency
-    - ed25519-dalek     # Internal eddsa implementation detail
-    - message_verification  # Requires quorum_validation
-    - quorum_validation # Requires BLS, complex setup
-    - rand-std          # Subset of rand
-    - secp-lowmemory    # Build optimization flag
-    - signer            # High-level feature, tested via all-features
+    - no-std                   # Broken, needs investigation
+    - bitcoinconsensus         # Requires native lib
+    - core-block-hash-use-x11  # Requires x11
+    - message_verification     # Requires quorum_validation feature
+  no-std: []  # dashcore no-std is broken
 
 dashcore-private:
-  skip:
-    - default           # Tested via --all-features
   no-std:
     - bare
     - alloc
 
 dash-network:
-  skip:
-    - default           # Tested via --all-features
-    - bincode           # Internal serialization
   no-std:
     - no-std
 
-key-wallet:
-  skip:
-    - default           # Tested via --all-features
-    - aes               # Internal dep for bip38
-    - base64            # Internal dep
-    - bincode           # Internal serialization
-    - bs58              # Internal dep
-    - getrandom         # Internal dep for rand
-    - rand              # Tested via all-features
-    - scrypt            # Internal dep for bip38
-
-key-wallet-manager:
-  skip:
-    - default           # Tested via --all-features
-    - bincode           # Internal serialization
-    - getrandom         # Internal dep
-
-key-wallet-ffi:
-  skip:
-    - default           # Tested via --all-features
-    - bincode           # Internal serialization
-
-dashcore-test-utils:
-  skip:
-    - tokio             # Internal dep for async
-
 dash-spv:
   skip:
-    - skip_mock_implementation_incomplete  # Dev flag
-    - terminal-ui       # Optional UI, not core functionality
+    - skip_mock_implementation_incomplete  # Dev flag, not a real feature