8000 revamp support level enumeration in component matcher by jctanner · Pull Request #1101 · ansible/ansibullbot · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on May 20, 2024. It is now read-only.

revamp support level enumeration in component matcher #1101

Merged
merged 7 commits into from
Nov 25, 2018
Merged

revamp support level enumeration in component matcher #1101

merged 7 commits into from
Nov 25, 2018

Conversation

jctanner
Copy link
Contributor
@jctanner jctanner commented Nov 21, 2018

Fixes #1099
tests TBD

@codecov
Copy link
codecov bot commented Nov 21, 2018

Codecov Report

Merging #1101 into master will increase coverage by 0.09%.
The diff coverage is 60%.

@@            Coverage Diff             @@
##           master    #1101      +/-   ##
==========================================
+ Coverage   31.36%   31.45%   +0.09%     
==========================================
  Files          47       47              
  Lines        9173     9195      +22     
==========================================
+ Hits         2877     2892      +15     
- Misses       6296     6303       +7

@gundalow
Copy link
Collaborator

Other examples

@jctanner jctanner changed the title WIP: Use more direct matching on the support level revamp support level enumeration in component matcher Nov 25, 2018
@jctanner jctanner merged commit efed320 into ansible:master Nov 25, 2018
# reconcile support levels
if filename in support_levels:
# exact match
meta['support'] = support_levels[filename]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

keys need u-prefix

@@ -430,6 +431,8 @@ def __init__(self, filepath, email_cache=None):
@property
def filedata(self):
if self._FILEDATA is None:
if not os.path.exists(self.filepath):
return ''
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

u

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0