Skip to content

Conversation

NULLx76
Copy link
Member

@NULLx76 NULLx76 commented Jun 15, 2019

No description provided.

@NULLx76
Copy link
Member Author

NULLx76 commented Jun 15, 2019

Blocked by #51

@codecov
Copy link

codecov bot commented Jun 17, 2019

Codecov Report

Merging #50 into develop will not change coverage.
The diff coverage is 75%.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop      #50   +/-   ##
========================================
  Coverage    62.75%   62.75%           
========================================
  Files           23       23           
  Lines         1893     1893           
  Branches       507      507           
========================================
  Hits          1188     1188           
  Misses         226      226           
  Partials       479      479
Impacted Files Coverage Δ
src/payloads/binmemberauthenticationpayload.rs 53.84% <ø> (ø) ⬆️
src/crypto/mod.rs 57.89% <0%> (ø) ⬆️
src/community/peer.rs 81.08% <100%> (ø) ⬆️
src/serialization/mod.rs 67.65% <100%> (ø) ⬆️
src/community/mod.rs 70.83% <70.31%> (ø) ⬆️
src/crypto/signature.rs 60.5% <71.42%> (ø) ⬆️
src/networking/mod.rs 71.42% <77.77%> (ø) ⬆️
src/crypto/keytypes.rs 57.08% <82.35%> (ø) ⬆️
... and 5 more

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 050fdbd...ae76899. Read the comment docs.

@codecov
Copy link

codecov bot commented Jun 17, 2019

Codecov Report

Merging #50 into develop will decrease coverage by 0.91%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #50      +/-   ##
===========================================
- Coverage    67.06%   66.15%   -0.92%     
===========================================
  Files           21       20       -1     
  Lines         1245     1179      -66     
  Branches       293      279      -14     
===========================================
- Hits           835      780      -55     
- Misses         128      139      +11     
+ Partials       282      260      -22
Impacted Files Coverage Δ
rust_ipv8/src/crypto/signature.rs 55.17% <0%> (-15.13%) ⬇️
rust_ipv8/src/payloads/puncturepayload.rs 72.22% <0%> (ø) ⬆️
rust_ipv8/src/serialization/rawend.rs 58.53% <0%> (ø) ⬆️
rust_ipv8/src/serialization/nestedpayload.rs 67.85% <0%> (ø) ⬆️
rust_ipv8/src/payloads/timedistributionpayload.rs 63.63% <0%> (ø) ⬆️
rust_ipv8/src/serialization/mod.rs 75.86% <0%> (ø) ⬆️
rust_ipv8/src/serialization/varlen.rs 53.22% <0%> (ø) ⬆️
rust_ipv8/src/community/peer.rs 86.95% <0%> (ø) ⬆️
rust_ipv8/src/community/mod.rs 89.32% <0%> (ø) ⬆️
rust_ipv8/src/serialization/bits.rs 66.32% <0%> (ø) ⬆️
... and 10 more

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 b42c5e5...7fa2ae1. Read the comment docs.

Some(self.peer.get_sha1()?.to_vec())
}

Choose a reason for hiding this comment

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

Expected an indentation at 8 instead of at 6.

Some(self.peer.get_sha1()?.to_vec())
}

Choose a reason for hiding this comment

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

Unexpected trailing spaces found.

Victor Roest added 3 commits June 17, 2019 13:04
# channels to enable unstable features when building for nightly. Or you could add additional
# matrix entries to test different combinations of features.
environment:
matrix:

Choose a reason for hiding this comment

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

Expected an indentation at 4 instead of at 2.


### MSVC Toolchains ###

# Stable 64-bit MSVC

Choose a reason for hiding this comment

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

Expected an indentation at 4 instead of at 2.


# Stable 64-bit MSVC
- channel: stable
target: x86_64-pc-windows-msvc

Choose a reason for hiding this comment

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

Expected an indentation at 8 instead of at 6.


### GNU Toolchains ###

# Stable 64-bit GNU

Choose a reason for hiding this comment

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

Expected an indentation at 4 instead of at 2.


# Stable 64-bit GNU
- channel: stable
target: x86_64-pc-windows-gnu

Choose a reason for hiding this comment

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

Expected an indentation at 8 instead of at 6.

# default-toolchain and default-host manually here.
install:
- appveyor DownloadFile https://win.rustup.rs/ -FileName rustup-init.exe
- rustup-init -yv --default-toolchain %channel% --default-host %target%

Choose a reason for hiding this comment

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

Expected an indentation at 4 instead of at 2.

install:
- appveyor DownloadFile https://win.rustup.rs/ -FileName rustup-init.exe
- rustup-init -yv --default-toolchain %channel% --default-host %target%
- set PATH=%PATH%;%USERPROFILE%\.cargo\bin

Choose a reason for hiding this comment

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

Expected an indentation at 4 instead of at 2.

- appveyor DownloadFile https://win.rustup.rs/ -FileName rustup-init.exe
- rustup-init -yv --default-toolchain %channel% --default-host %target%
- set PATH=%PATH%;%USERPROFILE%\.cargo\bin
- rustc -vV

Choose a reason for hiding this comment

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

Expected an indentation at 4 instead of at 2.

- rustup-init -yv --default-toolchain %channel% --default-host %target%
- set PATH=%PATH%;%USERPROFILE%\.cargo\bin
- rustc -vV
- cargo -vV

Choose a reason for hiding this comment

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

Expected an indentation at 4 instead of at 2.

#directly or perform other testing commands. Rust will automatically be placed in the PATH
# environment variable.
test_script:
- cargo test --verbose %cargoflags%

Choose a reason for hiding this comment

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

Expected an indentation at 4 instead of at 2.

# channels to enable unstable features when building for nightly. Or you could add additional
# matrix entries to test different combinations of features.
environment:
matrix:

Choose a reason for hiding this comment

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

Expected an indentation at 4 instead of at 2.


### MSVC Toolchains ###

# Stable 64-bit MSVC

Choose a reason for hiding this comment

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

Expected an indentation at 4 instead of at 2.


# Stable 64-bit MSVC
- channel: stable
target: x86_64-pc-windows-msvc

Choose a reason for hiding this comment

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

Expected an indentation at 8 instead of at 6.


### GNU Toolchains ###

# Stable 64-bit GNU

Choose a reason for hiding this comment

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

Expected an indentation at 4 instead of at 2.


# Stable 64-bit GNU
- channel: stable
target: x86_64-pc-windows-gnu

Choose a reason for hiding this comment

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

Expected an indentation at 8 instead of at 6.

# default-toolchain and default-host manually here.
install:
- appveyor DownloadFile https://win.rustup.rs/ -FileName rustup-init.exe
- rustup-init -yv --default-toolchain %channel% --default-host %target%

Choose a reason for hiding this comment

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

Expected an indentation at 4 instead of at 2.

install:
- appveyor DownloadFile https://win.rustup.rs/ -FileName rustup-init.exe
- rustup-init -yv --default-toolchain %channel% --default-host %target%
- set PATH=%PATH%;%USERPROFILE%\.cargo\bin

Choose a reason for hiding this comment

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

Expected an indentation at 4 instead of at 2.

- appveyor DownloadFile https://win.rustup.rs/ -FileName rustup-init.exe
- rustup-init -yv --default-toolchain %channel% --default-host %target%
- set PATH=%PATH%;%USERPROFILE%\.cargo\bin
- rustc -vV

Choose a reason for hiding this comment

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

Expected an indentation at 4 instead of at 2.

- rustup-init -yv --default-toolchain %channel% --default-host %target%
- set PATH=%PATH%;%USERPROFILE%\.cargo\bin
- rustc -vV
- cargo -vV

Choose a reason for hiding this comment

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

Expected an indentation at 4 instead of at 2.

#directly or perform other testing commands. Rust will automatically be placed in the PATH
# environment variable.
test_script:
- cargo test --verbose %cargoflags%

Choose a reason for hiding this comment

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

Expected an indentation at 4 instead of at 2.

#directly or perform other testing commands. Rust will automatically be placed in the PATH
# environment variable.
test_script:
- cd rust_ipv8

Choose a reason for hiding this comment

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

Expected an indentation at 4 instead of at 2.

test_script:
- cd rust_ipv8
- cargo test --verbose %cargoflags%
- cd rust_ipv8

Choose a reason for hiding this comment

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

Expected an indentation at 4 instead of at 2.

still not sure about travis vs. appveyor
@NULLx76
Copy link
Member Author

NULLx76 commented Jun 21, 2019

Windows still seems to be having networking problems

@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit 7fa2ae1 and detected 13 issues on this pull request.

Here's the issue category breakdown:

Category Count
Style 13

View more on Code Climate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant