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

Stargate #546

Closed
wants to merge 100 commits into from
Closed

Stargate #546

wants to merge 100 commits into from

Conversation

fedekunze
Copy link
Contributor

@fedekunze fedekunze commented Sep 28, 2020

Description

For contributor use:

  • 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.
  • Code follows the module structure standards.
  • Wrote unit and integration tests
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer

For admin use:

  • Added appropriate labels to PR (ex. WIP, R4R, docs, etc)
  • Reviewers assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

@fedekunze fedekunze self-assigned this Sep 30, 2020
fedekunze and others added 13 commits September 30, 2020 13:22
* stargate 2

* fix imports

* stargate 3

* update export

* proto tools

* commands fixes

* rename client context

* remove client binary

* revert script changes

* update ethermintd

* fixes

* fixes

* update account

* more updates

* third party proto files

* move proto-files
:

* fix crypto

* account proto

* keys

* update testnet command

* evm proto

* genesis state proto

* messages
@fedekunze fedekunze changed the title Stargate 1 Stargate Oct 9, 2020
@lgtm-com
Copy link

lgtm-com bot commented Jan 11, 2021

This pull request introduces 1 alert when merging 36b85ad into 52d6d71 - view on LGTM.com

new alerts:

  • 1 for Useless assignment to local variable

@lgtm-com
Copy link

lgtm-com bot commented Jan 11, 2021

This pull request introduces 1 alert when merging 08c93c6 into 9ecd264 - view on LGTM.com

new alerts:

  • 1 for Useless assignment to local variable

@fedekunze fedekunze changed the base branch from development to fedekunze/stargate-8 January 11, 2021 15:44
@lgtm-com
Copy link

lgtm-com bot commented Jan 30, 2021

This pull request fixes 1 alert when merging 5884ba0 into 02ea658 - view on LGTM.com

fixed alerts:

  • 1 for Useless assignment to local variable

@lgtm-com
Copy link

lgtm-com bot commented Jan 30, 2021

This pull request fixes 1 alert when merging e16081f into 02ea658 - view on LGTM.com

fixed alerts:

  • 1 for Useless assignment to local variable

* fixed invalid chainID, subscribe nil pointer dereference and empty keyring issue

* add keyring flag in start cmd

* update init script

* add amount back in `gentx` cli
@lgtm-com
Copy link

lgtm-com bot commented Feb 8, 2021

This pull request fixes 1 alert when merging bf2f2aa into 02ea658 - view on LGTM.com

fixed alerts:

  • 1 for Useless assignment to local variable

* push changes to cmd for rpcapi options

* add flag parse in server file

* fix lint
@lgtm-com
Copy link

lgtm-com bot commented Mar 12, 2021

This pull request fixes 1 alert when merging 8c30756 into 02ea658 - view on LGTM.com

fixed alerts:

  • 1 for Useless assignment to local variable

@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days-before-close if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale stale PRs that will be closed if no further action occurs label Apr 27, 2021
@github-actions github-actions bot closed this May 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
stale stale PRs that will be closed if no further action occurs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants