Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problem: goreleaser is using wrong Env for GitHub token (fixes #251) #252

Merged
merged 1 commit into from
Nov 6, 2020

Conversation

tomtau
Copy link
Contributor

@tomtau tomtau commented Nov 6, 2020

Solution: updated to use github token + commented out brews, as it may need to be updated manually
before permissioning is figured out

…-org-chain#251)

Solution: updated to use github token + commented out brews, as it may need to be updated manually
before permissioning is figured out
@codecov
Copy link

codecov bot commented Nov 6, 2020

Codecov Report

Merging #252 into master will increase coverage by 0.14%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #252      +/-   ##
==========================================
+ Coverage   27.58%   27.72%   +0.14%     
==========================================
  Files          12       32      +20     
  Lines         986     5792    +4806     
==========================================
+ Hits          272     1606    +1334     
- Misses        674     4012    +3338     
- Partials       40      174     +134     
Flag Coverage Δ
integration_tests 27.72% <ø> (?)
unit_tests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
x/gov/client/cli/parse.go 0.00% <0.00%> (-100.00%) ⬇️
app/coins.go 8.82% <0.00%> (-52.95%) ⬇️
x/staking/client/cli/utils.go 0.00% <0.00%> (-38.47%) ⬇️
x/bank/client/cli/tx.go 43.02% <0.00%> (-24.45%) ⬇️
config/config.go 45.83% <0.00%> (-20.84%) ⬇️
x/staking/client/cli/tx.go 28.65% <0.00%> (-13.07%) ⬇️
x/chainmain/module.go 63.63% <0.00%> (ø)
x/chainmain/keeper/keeper.go 59.31% <0.00%> (ø)
x/staking/module.go 64.78% <0.00%> (ø)
x/chainmain/client/cli/tx.go 0.00% <0.00%> (ø)
... and 20 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b45fdbb...beaec41. Read the comment docs.

@tomtau tomtau merged commit 88a364c into crypto-org-chain:master Nov 6, 2020
@tomtau tomtau deleted the fix/goreleaser-2 branch November 6, 2020 10:11
tomtau added a commit that referenced this pull request Nov 6, 2020
…252)

Solution: updated to use github token + commented out brews, as it may need to be updated manually
before permissioning is figured out
tomtau added a commit that referenced this pull request Nov 6, 2020
…252)

Solution: updated to use github token + commented out brews, as it may need to be updated manually
before permissioning is figured out
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants