Remove editable bitcoin receiver actions

[tomer-stripe]

Removes the ability to use New and Update for bitcoin receivers. Tests pass!

» go test ./...
ok  	github.com/stripe/stripe-go	0.176s
ok  	github.com/stripe/stripe-go/account	0.081s
ok  	github.com/stripe/stripe-go/applepaydomain	(cached)
ok  	github.com/stripe/stripe-go/balance	0.106s
ok  	github.com/stripe/stripe-go/bankaccount	0.129s
ok  	github.com/stripe/stripe-go/bitcoinreceiver	0.086s
ok  	github.com/stripe/stripe-go/bitcointransaction	(cached)
ok  	github.com/stripe/stripe-go/card	0.267s
ok  	github.com/stripe/stripe-go/charge	0.089s
ok  	github.com/stripe/stripe-go/client	0.035s
ok  	github.com/stripe/stripe-go/countryspec	(cached)
ok  	github.com/stripe/stripe-go/coupon	(cached)
ok  	github.com/stripe/stripe-go/customer	0.085s
ok  	github.com/stripe/stripe-go/discount	(cached)
ok  	github.com/stripe/stripe-go/dispute	0.072s
ok  	github.com/stripe/stripe-go/ephemeralkey	(cached)
ok  	github.com/stripe/stripe-go/event	(cached)
ok  	github.com/stripe/stripe-go/exchangerate	(cached)
ok  	github.com/stripe/stripe-go/fee	0.092s
ok  	github.com/stripe/stripe-go/feerefund	0.123s
ok  	github.com/stripe/stripe-go/filelink	(cached)
ok  	github.com/stripe/stripe-go/fileupload	(cached)
ok  	github.com/stripe/stripe-go/form	(cached)
ok  	github.com/stripe/stripe-go/invoice	0.127s
ok  	github.com/stripe/stripe-go/invoiceitem	0.074s
ok  	github.com/stripe/stripe-go/issuerfraudrecord	0.134s
ok  	github.com/stripe/stripe-go/issuing/authorization	0.162s
ok  	github.com/stripe/stripe-go/issuing/card	(cached)
ok  	github.com/stripe/stripe-go/issuing/cardholder	(cached)
ok  	github.com/stripe/stripe-go/issuing/dispute	0.160s
ok  	github.com/stripe/stripe-go/issuing/transaction	0.123s
ok  	github.com/stripe/stripe-go/loginlink	(cached)
ok  	github.com/stripe/stripe-go/order	0.157s
ok  	github.com/stripe/stripe-go/orderreturn	0.111s
ok  	github.com/stripe/stripe-go/paymentintent	0.131s
ok  	github.com/stripe/stripe-go/paymentsource	0.099s
ok  	github.com/stripe/stripe-go/payout	0.078s
ok  	github.com/stripe/stripe-go/plan	(cached)
ok  	github.com/stripe/stripe-go/product	(cached)
ok  	github.com/stripe/stripe-go/recipient	0.095s
ok  	github.com/stripe/stripe-go/recipienttransfer	(cached) [no tests to run]
ok  	github.com/stripe/stripe-go/refund	0.076s
ok  	github.com/stripe/stripe-go/reversal	0.089s
ok  	github.com/stripe/stripe-go/sigma/scheduledqueryrun	(cached)
ok  	github.com/stripe/stripe-go/sku	(cached)
ok  	github.com/stripe/stripe-go/source	(cached)
ok  	github.com/stripe/stripe-go/sourcetransaction	(cached)
ok  	github.com/stripe/stripe-go/sub	0.121s
ok  	github.com/stripe/stripe-go/subitem	(cached)
ok  	github.com/stripe/stripe-go/testing	(cached)
ok  	github.com/stripe/stripe-go/threedsecure	0.067s
ok  	github.com/stripe/stripe-go/token	0.081s
ok  	github.com/stripe/stripe-go/topup	0.062s
ok  	github.com/stripe/stripe-go/transfer	0.056s
ok  	github.com/stripe/stripe-go/usagerecord	(cached)
ok  	github.com/stripe/stripe-go/webhook	(cached)

[enugent-stripe]

LGTM

I haven't deployed to the Go library so I don't know if there's anything extra you need to concern yourself with on that front, but the code looks dope.

[brandur-stripe]

Can we remove BitcoinReceiverParams as well?

[tomer-stripe]

Good catch, doesn't look like we need that still.

[brandur-stripe]

Nice! And one more minor upfront comment that I try to give everyone at least once: we generally try to maintain better commit hygiene on the public repos than what you might see internally. Most of us who contribute regularly use git commit --amend and/or git rebase to squash commits/messages down into something cohesive before merge — and usually to one commit per PR.

You can also us the "squash" button when merging, but usually the commit messages turn out better when written as a unified whole by their original author.

[tomer-stripe]

That makes sense! Will squash the commits and re-push

[brandur-stripe]

Released as 41.0.0.