Skip to content
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

Build sample apps in CI #909

Merged
merged 9 commits into from
Apr 15, 2020
Merged

Build sample apps in CI #909

merged 9 commits into from
Apr 15, 2020

Conversation

optout21
Copy link
Contributor

@optout21 optout21 commented Apr 1, 2020

Description

To avoid breakage, build the sample apps in every CI. Script to build all sample apps. See #910 , #904 .

Testing instructions

./tools/samples-build

Types of changes

New script to build the sample apps.

  • New auxiliary feature

Checklist

  • Prefix PR title with [WIP] if necessary.
  • Add tests to cover changes as needed.
  • Update documentation as needed.
  • If there is a related Issue, mention it in the description (e.g. 'Fixes Adopt Zilliqa to bech32 address format #444 ).

@codecov
Copy link

codecov bot commented Apr 1, 2020

Codecov Report

Merging #909 into master will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #909      +/-   ##
==========================================
+ Coverage   92.51%   92.53%   +0.02%     
==========================================
  Files         418      418              
  Lines       12141    12162      +21     
==========================================
+ Hits        11232    11254      +22     
+ Misses        909      908       -1     
Impacted Files Coverage Δ
walletconsole/lib/Util.cpp 92.00% <0.00%> (-1.25%) ⬇️
src/IoTeX/Signer.h 100.00% <0.00%> (ø)
src/Filecoin/Entry.h 100.00% <0.00%> (ø)
src/IoTeX/Signer.cpp 100.00% <0.00%> (ø)
src/IoTeX/Staking.cpp 100.00% <0.00%> (ø)
src/Filecoin/Entry.cpp 100.00% <0.00%> (ø)
src/Filecoin/Signer.cpp 100.00% <0.00%> (ø)
src/Ethereum/ABI/Function.h 100.00% <0.00%> (ø)
src/interface/TWStoredKey.cpp 90.52% <0.00%> (+2.10%) ⬆️

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 72c8b9e...60d2296. Read the comment docs.

@optout21
Copy link
Contributor Author

optout21 commented Apr 1, 2020

Note: very small FIO test extension inlcuded to not create a separate small PR :)

@optout21 optout21 marked this pull request as ready for review April 1, 2020 09:49
@optout21 optout21 changed the title [WIP] Build sample apps in CI Build sample apps in CI Apr 1, 2020
@optout21 optout21 requested a review from hewigovens April 1, 2020 09:50
tools/samples-build Outdated Show resolved Hide resolved
@hewigovens hewigovens requested review from madcake and vikmeup April 3, 2020 15:19
@hewigovens
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@vikmeup
Copy link
Contributor

vikmeup commented Apr 15, 2020

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@hewigovens
Copy link
Contributor

checks are not refreshed

@vikmeup vikmeup merged commit e8ff9c7 into master Apr 15, 2020
@vikmeup vikmeup deleted the ar/samplebuild branch April 15, 2020 02:11
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.

Adopt Zilliqa to bech32 address format
4 participants