Skip to content

ArchesMisSync: check for arch.list <-> arches.desc missync#529

Merged
gentoo-bot merged 1 commit intopkgcore:masterfrom
arthurzam:check-arches
Jan 20, 2023
Merged

ArchesMisSync: check for arch.list <-> arches.desc missync#529
gentoo-bot merged 1 commit intopkgcore:masterfrom
arthurzam:check-arches

Conversation

@arthurzam
Copy link
Copy Markdown
Member

Resolves: #526

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 19, 2023

Codecov Report

Base: 80.86% // Head: 80.85% // Decreases project coverage by -0.01% ⚠️

Coverage data is based on head (72ec9a3) compared to base (ecaf9a2).
Patch coverage: 75.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #529      +/-   ##
==========================================
- Coverage   80.86%   80.85%   -0.01%     
==========================================
  Files          56       56              
  Lines        8200     8212      +12     
  Branches     1844     1847       +3     
==========================================
+ Hits         6631     6640       +9     
- Misses       1470     1473       +3     
  Partials       99       99              
Impacted Files Coverage Δ
src/pkgcheck/checks/profiles.py 63.91% <75.00%> (+0.37%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Comment thread src/pkgcheck/checks/profiles.py Outdated
Copy link
Copy Markdown
Contributor

@mgorny mgorny left a comment

Choose a reason for hiding this comment

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

LGTM, though sam is probably right about the name.

if unused_profile_dirs := available_profile_dirs - seen_profile_dirs:
yield UnusedProfileDirs(sorted(unused_profile_dirs))

if arches_desc := frozenset().union(*self.repo.config.arches_desc.values()):
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice trick. I'd have used itertools.chain() myself.

Resolves: pkgcore#526
Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
@gentoo-bot gentoo-bot merged commit 72ec9a3 into pkgcore:master Jan 20, 2023
@gentoo-bot gentoo-bot temporarily deployed to github-pages January 20, 2023 07:04 — with GitHub Actions Inactive
@arthurzam arthurzam deleted the check-arches branch January 20, 2023 07:12
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.

Check for arch.list <-> arches.desc missync

4 participants