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

chore: refine the format of default_manifest in collect.py #4258

Merged
merged 1 commit into from
Oct 22, 2024

Conversation

xiangce
Copy link
Contributor

@xiangce xiangce commented Oct 21, 2024

  • reorder and regroup the 'configs' that should be loaded in collection
  • use the full path to the Class for each 'configs'
  • tapstop=2 for the inline YAML
  • replace Services combiner in datasources with UnitFiles Parser
  • and reorder python imports with pre-commit

Signed-off-by: Xiangce Liu [email protected]

rh-pre-commit.version: 2.3.1
rh-pre-commit.check-secrets: ENABLED

All Pull Requests:

Check all that apply:

  • Have you followed the guidelines in our Contributing document, including the instructions about commit messages?
  • No Sensitive Data in this change?
  • Is this PR to correct an issue?
  • Is this PR an enhancement?

Complete Description of Additions/Changes:

Add your description here

- reorder and regroup the 'configs' that should be loaded in collection
- use the full path to the Class for each 'configs'
- tapstop=2 for the inline YAML
- replace Services combiner in datasources with UnitFiles Parser
- and reorder python imports with pre-commit

Signed-off-by: Xiangce Liu <[email protected]>

rh-pre-commit.version: 2.3.1
rh-pre-commit.check-secrets: ENABLED
@codecov-commenter
Copy link

codecov-commenter commented Oct 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.01%. Comparing base (78ab69f) to head (186fd28).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4258   +/-   ##
=======================================
  Coverage   77.01%   77.01%           
=======================================
  Files         761      761           
  Lines       41464    41471    +7     
  Branches     8763     8763           
=======================================
+ Hits        31932    31939    +7     
  Misses       8480     8480           
  Partials     1052     1052           
Flag Coverage Δ
unittests 77.00% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@chenlizhong chenlizhong left a comment

Choose a reason for hiding this comment

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

Looks good to me

@xiangce xiangce merged commit 28b64a1 into RedHatInsights:master Oct 22, 2024
15 checks passed
@xiangce xiangce deleted the collect_manifest branch October 22, 2024 07:50
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