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

Revise Inventory equality #8

Merged
merged 1 commit into from
Feb 23, 2024
Merged

Revise Inventory equality #8

merged 1 commit into from
Feb 23, 2024

Conversation

goerz
Copy link
Member

@goerz goerz commented Feb 23, 2024

The source and sorted attributes do not matter when comparing two Inventory objects. This makes it easier to load two inventory files and compare them.

On the other hand, root_url does matter, since that changes what e.g. the uri function returns.

@goerz goerz added the breaking PRs that break compatibility label Feb 23, 2024
The `source` and `sorted` attributes do not matter when comparing two
`Inventory` objects. This makes it easier to load two inventory files
and compare them.

On the other hand, `root_url` does matter, since that changes what e.g.
the `uri` function returns.
@goerz goerz merged commit cf1ffaf into master Feb 23, 2024
9 checks passed
@goerz goerz deleted the inventory-equality branch February 23, 2024 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking PRs that break compatibility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant