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

Allow osc rpmlint to infer prj/pkg from CWD. #1497

Merged
merged 1 commit into from
Feb 26, 2024
Merged

Allow osc rpmlint to infer prj/pkg from CWD. #1497

merged 1 commit into from
Feb 26, 2024

Conversation

badshah400
Copy link
Contributor

Makes osc rpmlint infer the project and package when working from a locally checked out package dir, if two or fewer arguments are passed. All four arguments are still required when called from a location that is not an osc package working copy.

This makes the behaviour of osc rpmlint similar to osc buildlogtail.

Makes `osc rpmlint` infer the project and package when working from a
locally checked out package dir, if two or fewer arguments are passed.
All four arguments are still required when called from a location that
is not an osc package working copy.

This makes the behaviour of `osc rpmlint` similar to `osc buildlogtail`.
Copy link

codecov bot commented Feb 26, 2024

Codecov Report

Attention: Patch coverage is 0% with 15 lines in your changes are missing coverage. Please review.

Project coverage is 31.41%. Comparing base (eab83d3) to head (78b07e6).
Report is 15 commits behind head on master.

Files Patch % Lines
osc/commandline.py 0.00% 15 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1497      +/-   ##
==========================================
- Coverage   31.51%   31.41%   -0.11%     
==========================================
  Files          51       71      +20     
  Lines       17945    18486     +541     
==========================================
+ Hits         5656     5807     +151     
- Misses      12289    12679     +390     

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

@dmach
Copy link
Contributor

dmach commented Feb 26, 2024

Looks good to me, merging.

@dmach dmach merged commit 58c3e2c into openSUSE:master Feb 26, 2024
31 of 35 checks passed
@badshah400
Copy link
Contributor Author

Thanks for the quick response and merge.

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.

2 participants