Merge pull request cosmos#10 from cosmos/master

Pull from master

.codecov.yml

@@ -24,3 +24,4 @@ ignore:
   - "docs"
   - "*.md"
   - "*.rst"
+  - "x/**/test_common.go"

.github/CODEOWNERS

@@ -1,9 +1,13 @@
 # CODEOWNERS: https://help.github.com/articles/about-codeowners/
 
 # Primary repo maintainers
-*       @ebuchman @rigelrozanski @cwgoes
+*       @rigelrozanski @alexanderbez @jackzampolin @alessio @fedekunze
 
-# Precious documentation
-/docs/README.md  @zramsay
-/docs/DOCS_README.md  @zramsay
-/docs/.vuepress/  @zramsay
+###############################################################################
+# Module Specific Ownership
+# See CONTRIBUTING.md for further details
+###############################################################################
+
+# The following contributors own all files in the x/nft directory at the root
+# of the repository and any of its subdirectories.
+x/nft @okwme @fedekunze

.github/ISSUE_TEMPLATE/bug-report.md

@@ -8,17 +8,25 @@ about: Create a report to help us squash bugs!
 v                            ✰  Thanks for opening an issue! ✰    
 v    Before smashing the submit button please review the template.
 v    Please also ensure that this is not a duplicate issue :)  
-☺ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >  --> 
+☺ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >  -->
 
-## Summary of Bug 
-<!-- Concisely describe the issue --> 
+## Summary of Bug
+
+<!-- Concisely describe the issue -->
+
+## Version
+
+<!-- git commit hash -->
 
 ## Steps to Reproduce
-<!-- What commands in order should someone run to reproduce your problem --> 
+
+<!-- What commands in order should someone run to reproduce your problem -->
 
 ____
+
 #### For Admin Use
- - [ ] Not duplicate issue
- - [ ] Appropriate labels applied
- - [ ] Appropriate contributors tagged
- - [ ] Contributor assigned/self-assigned
+
+- [ ] Not duplicate issue
+- [ ] Appropriate labels applied
+- [ ] Appropriate contributors tagged
+- [ ] Contributor assigned/self-assigned

.github/ISSUE_TEMPLATE/feature-request.md

@@ -9,23 +9,28 @@ v                            ✰  Thanks for opening an issue! ✰
 v    Before smashing the submit button please review the template.
 v    Word of caution: poorly thought-out proposals may be rejected 
 v                     without deliberation 
-☺ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >  --> 
+☺ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >  -->
 
 ## Summary
-<!-- Short, concise description of the proposed feature --> 
+
+<!-- Short, concise description of the proposed feature -->
 
 ## Problem Definition
+
 <!-- Why do we need this feature? 
 What problems may be addressed by introducing this feature?
 What benefits does the SDK stand to gain by including this feature?
 Are there any disadvantages of including this feature? -->
 
 ## Proposal
-<!-- Detailed description of requirements of implementation --> 
+
+<!-- Detailed description of requirements of implementation -->
 
 ____
+
 #### For Admin Use
- - [ ] Not duplicate issue
- - [ ] Appropriate labels applied
- - [ ] Appropriate contributors tagged
- - [ ] Contributor assigned/self-assigned
+
+- [ ] Not duplicate issue
+- [ ] Appropriate labels applied
+- [ ] Appropriate contributors tagged
+- [ ] Contributor assigned/self-assigned

.github/PULL_REQUEST_TEMPLATE.md

@@ -9,8 +9,8 @@ v    If a checkbox is n/a - please still include it but + a little note why
 - [ ] 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: `sdkch add [section] [stanza] [message]`
-- [ ] rereviewed `Files changed` in the github PR explorer
+- [ ] Added a relevant changelog entry to the `Unreleased` section in `CHANGELOG.md`
+- [ ] Re-reviewed `Files changed` in the github PR explorer
 
 ______
 

.github/stale.yml

@@ -0,0 +1,45 @@
+# Configuration for probot-stale - https://github.com/probot/stale
+
+# Number of days of inactivity before an Issue or Pull Request becomes stale
+daysUntilStale: 60
+
+# Number of days of inactivity before an Issue or Pull Request with the stale label is closed.
+# Set to false to disable. If disabled, issues still need to be closed manually, but will remain marked as stale.
+daysUntilClose: 9
+
+# Only issues or pull requests with all of these labels are check if stale. Defaults to `[]` (disabled)
+onlyLabels: []
+
+# Issues or Pull Requests with these labels will never be considered stale. Set to `[]` to disable
+exemptLabels:
+  - pinned
+  - security
+
+# Set to true to ignore issues in a project (defaults to false)
+exemptProjects: true
+
+# Set to true to ignore issues in a milestone (defaults to false)
+exemptMilestones: true
+
+# Label to use when marking as stale
+staleLabel: stale
+
+# Comment to post when marking as stale. Set to `false` to disable
+markComment: >
+  This issue has been automatically marked as stale because it has not had
+  recent activity. It will be closed if no further activity occurs. Thank you
+  for your contributions.
+
+# Limit the number of actions per hour, from 1-30. Default is 30
+limitPerRun: 30
+
+# Limit to only `issues` or `pulls`
+only: pulls
+
+# Optionally, specify configuration settings that are specific to just 'issues' or 'pulls':
+pulls:
+  daysUntilStale: 30
+  markComment: >
+    This pull request has been automatically marked as stale because it has not had
+    recent activity. It will be closed if no further activity occurs. Thank you
+    for your contributions.

.gitignore

@@ -7,28 +7,28 @@
 *.swn
 .vscode
 .idea
+*.pyc
 
 # Build
 vendor
-vendor-deps
-.vendor-new
 build
 tools/bin/*
 examples/build/*
 docs/_build
 docs/tutorial
+docs/node_modules
 dist
-devtools-stamp
+tools-stamp
 
 # Data - ideally these don't exist
 baseapp/data/*
 client/lcd/keys/*
-client/lcd/statik/statik.go
 mytestnet
 
 # Testing
 coverage.txt
 profile.out
+sim_log_file
 
 # Vagrant
 .vagrant/

.golangci.yml

@@ -0,0 +1,56 @@
+# run:
+#   # timeout for analysis, e.g. 30s, 5m, default is 1m
+#   timeout: 5m
+
+linters:
+  enable:
+    - bodyclose
+    - deadcode
+    - depguard
+    - dogsled
+    - goconst
+    - gocritic
+    - gofmt
+    - goimports
+    - golint
+    - gosec
+    - gosimple
+    - govet
+    - ineffassign
+    - interfacer
+    - maligned
+    - misspell
+    - nakedret
+    - prealloc
+    - scopelint
+    - staticcheck
+    - structcheck
+    - stylecheck
+    - typecheck
+    - unconvert
+    - unused
+    - misspell
+  disable:
+    - errcheck
+
+issues:
+  exclude-rules:
+    - text: "Use of weak random number generator"
+      linters:
+        - gosec
+    - text: "comment on exported var"
+      linters:
+        - golint
+    - text: "don't use an underscore in package name"
+      linters:
+        - golint
+    - text: "ST1003:"
+      linters:
+        - stylecheck
+
+linters-settings:
+  dogsled:
+    max-blank-identifiers: 3
+  maligned:
+    # print struct with more effective memory layout or not, false by default
+    suggest-new: true