Skip to content
This repository has been archived by the owner on Dec 21, 2022. It is now read-only.

open CI and fix UT #12

Merged
merged 20 commits into from
Apr 27, 2020
Merged

open CI and fix UT #12

merged 20 commits into from
Apr 27, 2020

Conversation

ylsGit
Copy link

@ylsGit ylsGit commented Apr 26, 2020

  • 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 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, ready-for-review, docs)
  • Reviewers Assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

@zhongqiuwood zhongqiuwood merged commit 323b178 into v0.37.9-patched Apr 27, 2020
@ylsGit ylsGit deleted the linsheng/ci branch April 27, 2020 09:25
zhongqiuwood added a commit that referenced this pull request Apr 27, 2020
* okchain patch

* Change system fee to gas fee

* remove minimum-gas-prices from the server/start.go

* fix backend default config

* Remove some meaningless comments and some default value of flags

* fix the mock test func

* set 	defaultMinGasPrices = "0.00000001okt"

* FIX: issue #47, remove param msdLimited in config/default_params.go

* add fee

* Update query.go

change client tips from cosmosxxx to okchainxxx

* open CI and fix UT (#12)

* Update config.yml

* udpate go mod

* Update config.yml

* fix UT

* fix UT

* fix UT

* fix UT

* fix UT

* fix UT

* fix UT

* fix UT

* fix slashing UT

* Update config.yml

* cancel TxMsgs limit, revert to version cosmos

* Update config.yml

* Update config.yml

* Update config.yml

* Update README.md

Co-authored-by: ylsGit <>
Co-authored-by: yezhigang <[email protected]>

* To fix okchain issue #70 && order tx not requiring fees (#10)

* set the Infinite Gas Meter for order tx

Co-authored-by: GeYiLi <[email protected]>

Co-authored-by: liuxiong <[email protected]>
Co-authored-by: yanwei.han <[email protected]>
Co-authored-by: qzhongwood <[email protected]>
Co-authored-by: MichaelWang <[email protected]>
Co-authored-by: [email protected] <[email protected]>
Co-authored-by: ZhiGang <[email protected]>
Co-authored-by: ylsGit <[email protected]>
Co-authored-by: yezhigang <[email protected]>
Co-authored-by: banishee <[email protected]>
Co-authored-by: GeYiLi <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants