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

Check should not report missing %pre and %post deps (RhBug:1543449) #1512

Merged
merged 2 commits into from Mar 5, 2020

Conversation

kontura
Copy link
Contributor

@kontura kontura commented Oct 23, 2019

@j-mracek
Copy link
Member

j-mracek commented Nov 8, 2019

I suggest that overloading requires_pre is not good idea.

@rh-atomic-bot
Copy link

☔ The latest upstream changes (presumably 5ccde9f) made this pull request unmergeable. Please resolve the merge conflicts.

@packit-as-a-service
Copy link

There was an error while running a copr build:

{'chroots': ['Such chroot is not available: fedora-29-x86_64']}

You can re-trigger build by adding a comment (/packit copr-build) into this pull request.

@j-mracek
Copy link
Member

j-mracek commented Mar 5, 2020

@rh-atomic-bot r+

@rh-atomic-bot
Copy link

📌 Commit ed3987f has been approved by j-mracek

@rh-atomic-bot
Copy link

⌛ Testing commit ed3987f with merge 67c5c67...

rh-atomic-bot pushed a commit that referenced this pull request Mar 5, 2020
Closes: #1512
Approved by: j-mracek
@rh-atomic-bot
Copy link

💔 Test failed - status-papr

@j-mracek j-mracek merged commit 38011c8 into rpm-software-management:master Mar 5, 2020
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.

None yet

3 participants