Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Add solana-fullnode-config #294

Merged
merged 3 commits into from
May 31, 2018

Conversation

garious
Copy link
Contributor

@garious garious commented May 31, 2018

No description provided.

@garious garious requested review from aeyakovenko and sakridge May 31, 2018 21:46
Copy link
Contributor

@sakridge sakridge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@garious garious merged commit 6ab0dd4 into solana-labs:master May 31, 2018
@codecov
Copy link

codecov bot commented May 31, 2018

Codecov Report

Merging #294 into master will decrease coverage by 0.67%.
The diff coverage is 42.55%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #294      +/-   ##
==========================================
- Coverage    88.5%   87.82%   -0.68%     
==========================================
  Files          32       32              
  Lines        2983     3030      +47     
==========================================
+ Hits         2640     2661      +21     
- Misses        343      369      +26
Impacted Files Coverage Δ
src/lib.rs 100% <ø> (ø) ⬆️
src/crdt.rs 67.32% <42.55%> (-2.85%) ⬇️
src/streamer.rs 82.58% <0%> (+0.26%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c9fd5d7...d208601. Read the comment docs.

yihau pushed a commit to yihau/solana that referenced this pull request Apr 3, 2024
* alt_bn128: simplify errors in sycalls (alt_bn128, compress, poseidon)

* add TODO for feature gate. remove validate from compress

* add feature gate

* fix one more error case

* all changes under feature gate

* revert removing from()

* return unexpected errors in lib

* add comment to remove error types, once the feature gate is activated

* remove unnecessary/impossible error

* fix mispelled comments
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants