Skip to content

Commit

Permalink
Add warnings key to build status (#16)
Browse files Browse the repository at this point in the history
  • Loading branch information
fauxpark authored Oct 12, 2023
1 parent 54834a6 commit 0c213e3
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions qmk_api_tasks.py
Original file line number Diff line number Diff line change
Expand Up @@ -270,7 +270,7 @@ def run(self):
output = f'No layouts for {keyboard}! Skipping!'
bad_boards += 1
qmk_redis.set('qmk_bad_boards', bad_boards)
configurator_build_status[keyboard] = {'works': False, 'last_tested': int(time()), 'message': output}
configurator_build_status[keyboard] = {'works': False, 'warnings': False, 'last_tested': int(time()), 'message': output}
keyboards_tested[keyboard] = False # FIXME: Remove this when it's no longer used
failed_keyboards[keyboard] = {'severity': 'error', 'message': output} # FIXME: Remove this when it's no longer used
print(output)
Expand Down Expand Up @@ -303,7 +303,7 @@ def run(self):
print('Compile job completed successfully!')
good_boards += 1
qmk_redis.set('qmk_good_boards', good_boards)
configurator_build_status[keyboard] = {'works': True, 'last_tested': int(time()), 'message': result['output']}
configurator_build_status[keyboard] = {'works': True, 'warnings': '[WARNING]' in result['output'], 'last_tested': int(time()), 'message': result['output']}
keyboards_tested[keyboard] = True # FIXME: Remove this when it's no longer used
if keyboard in failed_keyboards:
del failed_keyboards[keyboard] # FIXME: Remove this when it's no longer used
Expand All @@ -327,7 +327,7 @@ def run(self):

bad_boards += 1
qmk_redis.set('qmk_bad_boards', bad_boards)
configurator_build_status[keyboard] = {'works': False, 'last_tested': int(time()), 'message': output}
configurator_build_status[keyboard] = {'works': False, 'warnings': False, 'last_tested': int(time()), 'message': output}
keyboards_tested[keyboard] = False # FIXME: Remove this when it's no longer used
failed_keyboards[keyboard] = {'severity': 'error', 'message': output} # FIXME: Remove this when it's no longer used

Expand Down

0 comments on commit 0c213e3

Please sign in to comment.