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

Rename dto to inventory #13337

Merged
merged 1 commit into from
Jan 9, 2017
Merged

Conversation

Ladas
Copy link
Contributor

@Ladas Ladas commented Jan 3, 2017

Rename 'dto' to 'inventory', which is more understandable term

@Ladas Ladas force-pushed the rename_dto_to_inventory branch from 4ebc860 to 230afc2 Compare January 3, 2017 16:49
@Ladas
Copy link
Contributor Author

Ladas commented Jan 3, 2017

@miq-bot assign @agrare

@Ladas
Copy link
Contributor Author

Ladas commented Jan 3, 2017

@agrare just the rename we talked about

Rename dto to inventory, which is more understandable term
@Ladas Ladas force-pushed the rename_dto_to_inventory branch from f05d455 to 416f07f Compare January 4, 2017 11:01
@miq-bot
Copy link
Member

miq-bot commented Jan 4, 2017

Checked commit Ladas@416f07f with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1
17 files checked, 83 offenses detected

app/models/manager_refresh/graph/topological_sort.rb

app/models/manager_refresh/inventory_collection.rb

app/models/manager_refresh/inventory_collection/graph.rb

app/models/manager_refresh/inventory_object.rb

app/models/manager_refresh/inventory_object_lazy.rb

app/models/manager_refresh/save_collection/helper.rb

app/models/manager_refresh/save_collection/recursive.rb

app/models/manager_refresh/save_collection/topological_sort.rb

  • ❕ - Line 6, Col 7 - Metrics/AbcSize - Assignment Branch Condition size for save_collections is too high. [31.34/20]

app/models/manager_refresh/save_inventory.rb

spec/models/manager_refresh/save_inventory/acyclic_graph_of_inventory_collections_spec.rb

spec/models/manager_refresh/save_inventory/graph_of_inventory_collections_spec.rb

spec/models/manager_refresh/save_inventory/graph_of_inventory_collections_targeted_refresh_spec.rb

spec/models/manager_refresh/save_inventory/single_inventory_collection_spec.rb

@agrare
Copy link
Member

agrare commented Jan 9, 2017

LGTM

@agrare agrare merged commit 21e0f8f into ManageIQ:master Jan 9, 2017
@agrare agrare added the euwe/no label Jan 9, 2017
@agrare agrare added this to the Sprint 52 Ending Jan 16, 2017 milestone Jan 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants