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

refactor: Remove useless stub BeginBlock/EndBlock methods #853

Merged
merged 3 commits into from
Jan 5, 2023
Merged

refactor: Remove useless stub BeginBlock/EndBlock methods #853

merged 3 commits into from
Jan 5, 2023

Conversation

jaeseung-bae
Copy link
Contributor

@jaeseung-bae jaeseung-bae commented Jan 4, 2023

Remove useless stub BeginBlock/EndBlock methods

Description

closes: #796

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

@codecov
Copy link

codecov bot commented Jan 4, 2023

Codecov Report

Merging #853 (bcc943b) into main (06751bd) will decrease coverage by 0.01%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #853      +/-   ##
==========================================
- Coverage   63.09%   63.07%   -0.02%     
==========================================
  Files         869      868       -1     
  Lines      100299   100292       -7     
==========================================
- Hits        63280    63258      -22     
- Misses      33514    33534      +20     
+ Partials     3505     3500       -5     
Impacted Files Coverage Δ
x/ibc/testing/mock/mock.go 0.00% <ø> (ø)
x/token/validation.go 70.90% <0.00%> (-20.00%) ⬇️
x/collection/msgs.go 46.57% <0.00%> (-3.43%) ⬇️
x/token/msgs.go 50.95% <0.00%> (+4.56%) ⬆️

@jaeseung-bae jaeseung-bae marked this pull request as ready for review January 4, 2023 05:45
@tkxkd0159
Copy link
Member

tkxkd0159 commented Jan 4, 2023

You should apply new interface for module which implement BeginBlock or EndBlock (e.g. capability)

@jaeseung-bae jaeseung-bae merged commit a3d51b9 into Finschia:main Jan 5, 2023
zemyblue added a commit to zemyblue/finschia-sdk that referenced this pull request Jan 6, 2023
* main:
  refactor: Remove useless stub BeginBlock/EndBlock methods (Finschia#853)
zemyblue added a commit to zemyblue/finschia-sdk that referenced this pull request Jan 12, 2023
* main: (30 commits)
  chore(deps): Bump actions/cache from 3.2.2 to 3.2.3 (Finschia#860)
  chore(deps): Bump golang.org/x/crypto from 0.4.0 to 0.5.0 (Finschia#854)
  feat: Remove `x/wasm` module (Finschia#850)
  refactor: Remove useless stub BeginBlock/EndBlock methods (Finschia#853)
  feat: enable querying based on mempool status (only gRPC) (Finschia#840)
  feat(x/foundation): remove unnecessary gov-mint feature (Finschia#848)
  chore(deps): Bump actions/cache from 3.2.1 to 3.2.2 (Finschia#845)
  chore(deps): Bump github.com/mattn/go-isatty from 0.0.16 to 0.0.17 (Finschia#847)
  chore(deps): Bump actions/cache from 3.2.0 to 3.2.1 (Finschia#841)
  fix: apply foundation audit (Finschia#834)
  chore(deps): Bump actions/setup-go from 3.4.0 to 3.5.0 (Finschia#831)
  chore(deps): Bump actions/cache from 3.0.11 to 3.2.0 (Finschia#839)
  ci: automate release process (Finschia#829)
  chore(deps): Bump github.com/prometheus/common from 0.37.0 to 0.39.0 (Finschia#832)
  chore(deps): Bump goreleaser/goreleaser-action from 3 to 4 (Finschia#830)
  chore(deps): Bump github.com/magiconair/properties from 1.8.6 to 1.8.7 (Finschia#826)
  chore(deps): Bump technote-space/get-diff-action from 6.1.1 to 6.1.2 (Finschia#822)
  chore(deps): Bump golang.org/x/crypto from 0.2.0 to 0.4.0 (Finschia#828)
  feat: Get validator pubkey considering KMS (Finschia#821)
  Remove support for composite (BLS) key type (Finschia#817)
  ...

# Conflicts:
#	x/foundation/msgs.go
#	x/wasm/lbmtypes/codec.go
#	x/wasm/types/codec.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove stub BeginBlocker and EndBlockers in the original modules
3 participants