Skip to content
This repository has been archived by the owner on Nov 30, 2021. It is now read-only.

evm: move Keeper and Querier to /keeper package #182

Merged
merged 7 commits into from
Mar 9, 2020
Merged

evm: move Keeper and Querier to /keeper package #182

merged 7 commits into from
Mar 9, 2020

Conversation

fedekunze
Copy link
Contributor

closes #181

  • Targeted PR against correct branch (see CONTRIBUTING.md)

  • Linked to github-issue with discussion and accepted design OR link to spec that describes this work.

  • Wrote tests

  • Updated relevant documentation (docs/)

  • Added entries in PENDING.md with issue #

  • rereviewed Files changed in the github PR explorer

For Admin Use:

  • Added appropriate labels to PR (ex. wip, ready-for-review, docs)
  • Reviewers Assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]"

@fedekunze fedekunze added the x/evm EVM module issues label Mar 6, 2020
@CLAassistant
Copy link

CLAassistant commented Mar 6, 2020 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

austinabell
austinabell approved these changes Mar 6, 2020
View reviewed changes
Copy link

@austinabell austinabell left a comment

Choose a reason for hiding this comment

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

Awesome, I had noticed the recommended structure had changed since I had setup this and just forgot to refactor later

x/evm/keeper/keeper.go Show resolved Hide resolved
x/evm/keeper/keeper.go Show resolved Hide resolved
@fedekunze fedekunze merged commit 30f34e6 into development Mar 9, 2020
@fedekunze fedekunze deleted the fedekunze/181-update-module-structure branch June 8, 2020 09:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@austinabell austinabell austinabell approved these changes

Assignees
No one assigned
Labels
x/evm EVM module issues
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update x/evm to recommended module structure
3 participants
@fedekunze @CLAassistant @austinabell