-
-
Notifications
You must be signed in to change notification settings - Fork 24
Testing, linting, formating. Oh my! #36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Testing, linting, formating. Oh my! #36
Conversation
69b507f to
6ac1364
Compare
|
Well then, we officially can't merge any of the other pull requests anymore 😆 |
Most will still be fine, just a quick update from upstream should fix them |
Alright, we should merge this first as high priority, can we make sure everything works and we'll just merge it? |
This should be last still and then run linting and formatting at the end of current queue |
|
Although space indent 2 I'm not sure I agree with, 4 is definitely a standard already |
Fair, I think regardless of the order things should start moving or we'll get in more and more of a hole lol |
Core
Testing
tests/messaging.test.js- P2P messaging layer teststests/security.test.js- PoW and Ed25519 signature teststests/state.test.js- Bloom filter, HyperLogLog, LRU cache, and peer state testsscripts/test-network.js- Tests server P2P connection.github/workflows/test.yml) for automated test runsCode Quality
.editorconfigfor cross-editor consistencyDevEx
extensions.jsonwith recommended extensionssettings.jsonwith project-specific settingsCONTRIBUTING.md