fedora-qa / fedfind

Clone
Source Code
GIT

#12 'Simplify' cid_from_label
Closed 6 years ago by adamwill. Opened 6 years ago by adamwill.
demo-broken  into  master

file modified
+1 -6 
@@ -559,12 +559,7 @@ 
 

      labels, so there's really no option but to require the user to
 

      specify. Uses PDC.
 

      """
 

-     short = short.lower()
 

-     params = {'release': "{0}-{1}".format(short, release), 'compose_label': label}
 

-     res = pdc_query('composes', params)
 

-     if res:
 

-         return res[0]['compose_id']
 

-     return ''
 

+     return 'thisisnotacid'
 

  

  def label_from_cid(cid):
 

      """Get the compose label for a compose by ID. Only completed

file modified
+1 -1 
@@ -1,5 +1,5 @@ 
 

  [tox]
 

- envlist = py26,py27,py34,py35,py36,py37
 

+ envlist = py27,py34,py35,py36,py37
 

  skip_missing_interpreters=true
 

  [testenv]
 

  deps=py27,py34,py35,py36,py37: -r{toxinidir}/install.requires

This is shorter! It must be better!

(This is an intentionally broken commit to be used for a talk
demo, DO NOT MERGE).

Signed-off-by: Adam Williamson awilliam@redhat.com

1 new commit added

  • Disable Python 2.6 testing for demo purposes
6 years ago

Pull-Request has been closed by adamwill
6 years ago
Changes Summary 2
+1 -6
file changed
fedfind/helpers.py
+1 -1
file changed
tox.ini
Powered by Pagure 5.13.3
Documentation • File an Issue • About • SSH Hostkey/Fingerprint
© Red Hat, Inc. and others.