Skip to content
This repository has been archived by the owner on Jun 3, 2024. It is now read-only.

Prevent loops in the dependency graph #78

Open
Tiagoperes opened this issue Jan 13, 2023 · 0 comments
Open

Prevent loops in the dependency graph #78

Tiagoperes opened this issue Jan 13, 2023 · 0 comments
Labels
backlog an item of the backlog bug Something isn't working core Related only to the core library
Milestone

Comments

@Tiagoperes
Copy link
Contributor

There's nothing testing for cycles in the dependency graph. What happens when a cycle is detected? Does it start an infinite loop? Does the second appearance of an object gets ignored?

  1. Write a test
  2. Ignore the second time an object appears if it's currently entering an infinite loop.

Attention: the entries Nimbus itself creates in the dependency graph will never have cycles, but since this is an open API and new nodes can be added from anywhere, we should be able to detect possible errors introduced by the user.

@Tiagoperes Tiagoperes added bug Something isn't working backlog an item of the backlog core Related only to the core library labels Jan 13, 2023
@Tiagoperes Tiagoperes added this to the 1.0.0 milestone Jan 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backlog an item of the backlog bug Something isn't working core Related only to the core library
Projects
Status: 📋 Backlog
Development

No branches or pull requests

1 participant