R4R: Remove redundant action tag in x/gov

[hendrikhofstadt]

The transaction action tag is already set in runMsgs of baseapp.go. Ref: #2943

cosmos-sdk/baseapp/baseapp.go

Line 634 in f65ae49

tags = append(tags, sdk.MakeTag(sdk.TagAction, []byte(msg.Type())))

Redundant tags make it hard to distinguish between results of multiple messages in a transaction.

CC: @alexanderbez (#3091)

---

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)

[codecov]

Codecov Report

Merging #3328 into develop will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           develop   #3328      +/-   ##
==========================================
- Coverage    55.63%   55.6%   -0.03%     
==========================================
  Files          132     132              
  Lines         9559    9556       -3     
==========================================
- Hits          5318    5314       -4     
- Misses        3905    3906       +1     
  Partials       336     336

[fedekunze]

utACK

[alexanderbez]

Thanks @SLAMPER -- this indeed makes sense to me.

[jackzampolin]

Thank you for the PR @SLAMPER! This is a good catch.