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

Remove rune v3 #36

Merged
merged 7 commits into from Oct 6, 2018
Merged

Remove rune v3 #36

merged 7 commits into from Oct 6, 2018

Conversation

josephyi
Copy link
Owner

@josephyi josephyi commented Oct 6, 2018

No description provided.

Joseph Yi added 7 commits May 9, 2017 11:29
Feels good to delete old code
Feels bad for delay
There's still much more to update
Hope to finish soon
how does this even happen
reset head force push
Keystones have taken over
I guess less is more
let (:url2) {expand_template(Taric::Operation::LolStaticData::STATIC_VERSIONS.template_url)}
before {stub_get_json(url2, 'static_versions.json')}
let (:url2) {expand_template(Taric::Operation::Champion::CHAMPIONS.template_url, region: 'na')}
before {stub_get(url2).to_return(body: fixture('champions.json'), headers: {content_type: 'application/json; charset=utf-8'})}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Layout/SpaceInsideBlockBraces: Space missing inside {.
Style/StringLiterals: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
Metrics/LineLength: Line is too long. [132/80]
Layout/SpaceInsideHashLiteralBraces: Space inside { missing.
Layout/SpaceInsideHashLiteralBraces: Space inside } missing.
Layout/SpaceInsideBlockBraces: Space missing inside }.

@@ -41,27 +41,27 @@
describe '#execute!' do
let (:url) {expand_template(Taric::Operation::LolStatus::SHARD.template_url, region: 'na')}
before {stub_get(url).to_return(body: fixture('shard.json'), headers: {content_type: 'application/json; charset=utf-8'})}
let (:url2) {expand_template(Taric::Operation::LolStaticData::STATIC_VERSIONS.template_url)}
before {stub_get_json(url2, 'static_versions.json')}
let (:url2) {expand_template(Taric::Operation::Champion::CHAMPIONS.template_url, region: 'na')}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lint/ParenthesesAsGroupedExpression: (...) interpreted as grouped expression.
Layout/SpaceInsideBlockBraces: Space missing inside {.
Metrics/LineLength: Line is too long. [101/80]
Style/StringLiterals: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
Layout/SpaceInsideBlockBraces: Space missing inside }.

@@ -1,3 +1,3 @@
module Taric
VERSION = '1.0.0'
VERSION = '2.0.0-alpha'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/MutableConstant: Freeze mutable objects assigned to constants.
Style/StringLiterals: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.

@josephyi josephyi closed this Oct 6, 2018
@josephyi josephyi reopened this Oct 6, 2018
@josephyi josephyi merged commit abf8daa into master Oct 6, 2018
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling bac8c54 on remove_rune_v3 into a50fb6c on master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling bac8c54 on remove_rune_v3 into a50fb6c on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling bac8c54 on remove_rune_v3 into a50fb6c on master.

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.

None yet

3 participants