ManageIQ / ManageIQ/inventory_refresh
Move Scanner.scan under TopologicalSort
Open
Nobody has claimed this yet.
cleanup
help wanted
pinned
size/s
- Dominant language
- Ruby
- Stars
- 1
- Forks
- 21
- PR merge metrics
- No merged PRs in 30d
Description
When there were multiple strategies (topological and recursive) it made sense to have scanning separate but now it can live under topological_sort
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating Scanner.scan and the topological_sort implementation. Read how scanning is currently invoked and how TopologicalSort is structured, then move the scan behavior under TopologicalSort while preserving existing behavior. Done means Scanner.scan is no longer separate and topological scanning still works.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- ruby
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100