stripe / stripe-ruby

94%

master: 97%

LAST BUILD ON BRANCH brandur-frozen-strings

branch: brandur-frozen-strings

CHANGE BRANCH

x

Reset

• brandur-frozen-strings
• HEAD
• akropp-add-client-telemetry
• akropp-telemetry-in-ms
• alexander/flexible-billing
• anniel-coveralls
• anniel-test-coverage
• beta
• brandur-assert-raises
• brandur-backwards-auto-paging-each
• brandur-check-request-id
• brandur-clean-test-output
• brandur-clearable-connections
• brandur-configuration-ca-bundle-path
• brandur-configuring-version
• brandur-connection-manager-gc
• brandur-deleted-objects-with-ids
• brandur-disable-lengths
• brandur-drop-old-ruby
• brandur-drop-timecop
• brandur-exclude-examples
• brandur-expand-retries
• brandur-expose-compute-signature
• brandur-fix-delete-discount
• brandur-fix-query-encoding
• brandur-fix-tests
• brandur-generate-header
• brandur-hash
• brandur-idempotent-replayed
• brandur-initialize-instance-var
• brandur-integer-indexes
• brandur-keep-alive-to-30
• brandur-merge-query-params
• brandur-minor-refactor
• brandur-minor-stripe-client-cleanup
• brandur-minor-test-fixes
• brandur-monotonic-time
• brandur-named-code
• brandur-net-http
• brandur-net-http-persistent
• brandur-nicer-error
• brandur-only-empty-metadata
• brandur-port-in-output
• brandur-raise-argument-error
• brandur-relax-file-constraints
• brandur-remove-deprecated
• brandur-remove-file-encoding
• brandur-rename-request
• brandur-request-begin
• brandur-request-id-test
• brandur-retry-fix
• brandur-retry-lock-timeouts
• brandur-rubocop-080
• brandur-ruby-20
• brandur-ruby-21
• brandur-ruby-25
• brandur-select-port
• brandur-shoulda-context-2-0-0-rc4
• brandur-simpler-encoding
• brandur-stripe-mock-0-15
• brandur-stripe-mock-030
• brandur-stripe-mock-v0.12.0
• brandur-stripe-should-retry
• brandur-support-express-authorize
• brandur-thread-state
• brandur-warn-on-remove-error
• brandur-webmock-380
• cjavilla/add-playlist-link-to-readme
• cjavilla/update-readme
• ctrudeau-add-code-generated-comment-to-ruby
• daz-terminal-bindings
• fay/issfr
• fay/update-version
• foresthu-add-topup-cancel-support
• integration-v5
• jkakar/fix-product-issues
• jkakar/topup-client
• latest-codegen
• master
• metadata-api
• mickjermsurawong/update-stripe-mock-version
• ob-better-resource-url
• ob-bump-faraday
• ob-bump-rubocop
• ob-creditnote-preview
• ob-decimal-string
• ob-dsl-custom-method
• ob-early-fraud-warning
• ob-enable-telemetry
• ob-error-code
• ob-error-object
• ob-file-duck-typing
• ob-file-link
• ob-file-resource
• ob-fix-ci
• ob-fix-tests
• ob-fix-windows
• ob-gem-metadata
• ob-github-actions
• ob-irc-badge
• ob-oauth
• ob-openapi-files
• ob-prep-file
• ob-proxy
• ob-remove-all
• ob-remove-issuerfraudrecord
• ob-remove-old-methods
• ob-rubocop
• ob-serialize-individual
• ob-skip-to-hash-for-nil
• ob-static-methods
• ob-stripe-mock-0.11.0
• ob-test-file-link-data
• ob-topup-cancel
• ob-travis-bundler
• ob-travis-ruby26
• ob-upcoming-line-items
• ob-upgrade-stripe-mock
• ob-vscode
• ob/codegen-40a20cd
• ob/codegen-63fe31a
• ob/codegen-ab6898a
• ob/codegen-ba4dcd0
• ob/codegen-c806fa1
• pavel/codegen
• pavel/more-inheritance
• pavel/more-method-sorting
• pavel/normalize-opts
• pavel/simplify-inheritance
• pavel/single-quotes
• ralex/codegen
• ralex/codegen-2
• ralex/fix-initialize-from
• ralex/list-urs-and-st
• ralex/unified_requests
• rattrayalex-stripe-patch-1
• rattrayalex/codegen-c806fa1
• rattrayalex/codegen-f7fc004
• remi-add-account-capabilities
• remi-add-account-links
• remi-add-checkout-session-retrieve
• remi-add-checkout-sessions
• remi-add-credit-notes
• remi-add-creditnote
• remi-add-customer-balance-transactions
• remi-add-invoice-methods
• remi-add-issuing-resources
• remi-add-mandate
• remi-add-payment-intent
• remi-add-payment-methods
• remi-add-person
• remi-add-radar-lists
• remi-add-radar-review
• remi-add-reporting-resources
• remi-add-sigma-scheduled-query-run
• remi-add-subscription-schedules
• remi-add-tax-ids
• remi-add-tax-rates
• remi-add-terminal-delete
• remi-add-usage-record-summary
• remi-add-webhook-endpoint
• remi-bring-nested-tests-up
• remi-change-readme
• remi-fix-bt
• remi-fix-checkout-sessions
• remi-fix-payment-intent
• remi-fix-readme
• remi-fix-tests
• remi-fix-tests-latest
• remi-move-payment-intent-stripe-mock
• remi-scheduled-run-mock
• remi-setupintents
• remi-wip-creditnote-line
• remi/codegen-3854362
• remi/codegen-474461f
• remi/codegen-4bd4c01
• remi/codegen-72527e3
• remi/codegen-824d494
• remi/codegen-d663cdb
• remi/codegen-f71053e
• remi/codegen-f75fd88
• richardm-apiref-link
• richardm-billingportalconfig
• richardm-generate-via-stripetype
• richardm-gha
• richardm-junit-output
• rubocop-0.79
• rwz/inline-paths
• rwz/sort-methods
• sdk-release/next-major
• tmaxwell/fix-connect-base
• tomer-remove-edittable-bitcoin-receivers-actions
• v3.10.0
• v3.11.0
• v3.12.0
• v3.12.1
• v3.13.0
• v3.13.1
• v3.14.0
• v3.15.0
• v3.16.0
• v3.17.0
• v3.17.1
• v3.17.2
• v3.18.0
• v3.19.0
• v3.20.0
• v3.21.0
• v3.22.0
• v3.23.0
• v3.24.0
• v3.25.0
• v3.26.0
• v3.26.1
• v3.27.0
• v3.28.0
• v3.29.0
• v3.30.0
• v3.31.0
• v3.31.1
• v3.9.1
• v3.9.2
• v4.0.0
• v4.0.1
• v4.0.2
• v4.0.3
• v4.1.0
• v4.10.0
• v4.11.0
• v4.12.0
• v4.13.0
• v4.14.0
• v4.15.0
• v4.16.0
• v4.17.0
• v4.18.0
• v4.18.1
• v4.19.0
• v4.2.0
• v4.20.0
• v4.20.1
• v4.21.0
• v4.21.1
• v4.21.2
• v4.21.3
• v4.22.0
• v4.22.1
• v4.23.0
• v4.3.0
• v4.4.0
• v4.4.1
• v4.5.0
• v4.6.0
• v4.7.0
• v4.7.1
• v4.8.0
• v4.8.1
• v4.9.0
• v4.9.1
• v5.0.0
• v5.0.1
• v5.1.0
• v5.1.1
• v5.10.0
• v5.11.0
• v5.12.0
• v5.12.1
• v5.13.0
• v5.14.0
• v5.15.0
• v5.16.0
• v5.17.0
• v5.18.0
• v5.19.0
• v5.2.0
• v5.20.0
• v5.21.0
• v5.22.0
• v5.23.0
• v5.23.1
• v5.24.0
• v5.25.0
• v5.26.0
• v5.27.0
• v5.28.0
• v5.29.0
• v5.29.1
• v5.3.0
• v5.30.0
• v5.4.0
• v5.4.1
• v5.5.0
• v5.6.0
• v5.7.0
• v5.7.1
• v5.8.0
• v5.9.0
• zwick_set_app_info_partner_updates

Build # 2354

Build Type

push

travis-ci

Committed by brandur

Commit Message

Add `frozen_string_literal` to every file and enforce Rubocop rule

Adds the magic `frozen_string_literal: true` comment to every file and
enables a Rubocop rule to make sure that it's always going to be there
going forward as well.

See here for more background [1], but the basic idea is that unlike many
other languages, static strings in code are mutable by default. This has
since been acknowledged as not a particularly good idea, and the
intention is to rectify the mistake when Ruby 3 comes out, where all
string literals will be frozen. The `frozen_string_literal` magic
comment was introduced in Ruby 2.3 as a way of easing the transition,
and allows libraries and projects to freeze their literals in advance.

I don't think this is breaking in any way: it's possible that users
might've been pulling out one of are literals somehow and mutating it,
but that would probably not have been useful for anything and would
certainly not be recommended, so I'm quite comfortable pushing this
change through as a minor version.

As discussed in #641.

[1] https://stackoverflow.com/a/37799399

Run Details

1488 of 1588 relevant lines covered (93.7%)

1037.79 hits per line