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: update release pipeline to match non-wasm env #901

Merged
merged 6 commits into from
Feb 14, 2023

Conversation

tkxkd0159
Copy link
Member

@tkxkd0159 tkxkd0159 commented Feb 14, 2023

Description

closes: #893

Motivation and context

How has this been tested?

Screenshots (if appropriate):

Checklist:

  • I followed the contributing guidelines and code of conduct.
  • I have added a relevant changelog to CHANGELOG.md
  • I have added tests to cover my changes.
  • I have updated the documentation accordingly.
  • I have updated API documentation client/docs/swagger-ui/swagger.yaml

Copy link
Member

@zemyblue zemyblue left a comment

Choose a reason for hiding this comment

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

Please add changelog :)

@tkxkd0159 tkxkd0159 changed the title ci: update release pipeline to match non-wasm env build: update release pipeline to match non-wasm env Feb 14, 2023
@codecov
Copy link

codecov bot commented Feb 14, 2023

Codecov Report

Merging #901 (9caaa99) into main (302cb11) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #901      +/-   ##
==========================================
- Coverage   62.33%   62.32%   -0.01%     
==========================================
  Files         653      653              
  Lines       79467    79443      -24     
==========================================
- Hits        49536    49516      -20     
+ Misses      27252    27248       -4     
  Partials     2679     2679              
Impacted Files Coverage Δ
simapp/test_helpers.go 9.96% <ø> (+0.10%) ⬆️
simapp/simd/cmd/root.go 87.92% <100.00%> (-1.12%) ⬇️
crypto/keys/internal/ecdsa/privkey.go 83.01% <0.00%> (+1.88%) ⬆️

@tkxkd0159 tkxkd0159 self-assigned this Feb 14, 2023
@tkxkd0159 tkxkd0159 merged commit a642968 into Finschia:main Feb 14, 2023
@tkxkd0159 tkxkd0159 deleted the update-release-ci branch February 14, 2023 16:39
zemyblue pushed a commit to zemyblue/finschia-sdk that referenced this pull request Mar 17, 2023
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.

Remove the codes and scripts related wasm
3 participants