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

Code reduction #390

Closed

Conversation

j-mracek
Copy link
Member

No description provided.

@j-mracek
Copy link
Member Author

j-mracek commented Nov 30, 2017

The pull-request cannot be released before New Year 2018.

@j-mracek
Copy link
Member Author

@rh-atomic-bot try

@rh-atomic-bot
Copy link

⌛ Trying commit b3e0443 with merge aef2afa...

rh-atomic-bot pushed a commit that referenced this pull request Nov 30, 2017
The functionality is unused and never was used.

Closes: #390
Approved by: <try>
rh-atomic-bot pushed a commit that referenced this pull request Nov 30, 2017
The functionality can be replaced by nevra_possibilities() and then test it with
nevra_to_query().

Closes: #390
Approved by: <try>
rh-atomic-bot pushed a commit that referenced this pull request Nov 30, 2017
The code is unused. It is recommend to use directly queries.

Closes: #390
Approved by: <try>
rh-atomic-bot pushed a commit that referenced this pull request Nov 30, 2017
Closes: #390
Approved by: <try>
rh-atomic-bot pushed a commit that referenced this pull request Nov 30, 2017
rh-atomic-bot pushed a commit that referenced this pull request Nov 30, 2017
Closes: #390
Approved by: <try>
@rh-atomic-bot
Copy link

💔 Test failed - status-papr

@j-mracek
Copy link
Member Author

@rh-atomic-bot try

@rh-atomic-bot
Copy link

⌛ Trying commit e737576 with merge 9443a53...

rh-atomic-bot pushed a commit that referenced this pull request Nov 30, 2017
The functionality is unused and never was used.

Closes: #390
Approved by: <try>
rh-atomic-bot pushed a commit that referenced this pull request Nov 30, 2017
The functionality can be replaced by nevra_possibilities() and then test it with
nevra_to_query().

Closes: #390
Approved by: <try>
rh-atomic-bot pushed a commit that referenced this pull request Nov 30, 2017
The code is unused. It is recommend to use directly queries.

Closes: #390
Approved by: <try>
rh-atomic-bot pushed a commit that referenced this pull request Nov 30, 2017
Closes: #390
Approved by: <try>
rh-atomic-bot pushed a commit that referenced this pull request Nov 30, 2017
@rh-atomic-bot
Copy link

☀️ Test successful - status-papr
State: approved= try=True

@j-mracek j-mracek requested review from hughsie and removed request for hughsie November 30, 2017 18:58
@jrohel
Copy link
Contributor

jrohel commented Dec 6, 2017

@rh-atomic-bot r+

@rh-atomic-bot
Copy link

📌 Commit acd3c12 has been approved by jrohel

@rh-atomic-bot
Copy link

⌛ Testing commit acd3c12 with merge 5514d54...

rh-atomic-bot pushed a commit that referenced this pull request Dec 6, 2017
The functionality can be replaced by nevra_possibilities() and then test it with
nevra_to_query().

Closes: #390
Approved by: jrohel
rh-atomic-bot pushed a commit that referenced this pull request Dec 6, 2017
The code is unused. It is recommend to use directly queries.

Closes: #390
Approved by: jrohel
rh-atomic-bot pushed a commit that referenced this pull request Dec 6, 2017
Closes: #390
Approved by: jrohel
rh-atomic-bot pushed a commit that referenced this pull request Dec 6, 2017
rh-atomic-bot pushed a commit that referenced this pull request Dec 6, 2017
Closes: #390
Approved by: jrohel
rh-atomic-bot pushed a commit that referenced this pull request Dec 6, 2017
Closes: #390
Approved by: jrohel
@rh-atomic-bot
Copy link

☀️ Test successful - status-papr
Approved by: jrohel
Pushing 5514d54 to master...

@jrohel jrohel self-assigned this Dec 6, 2017
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