-
Notifications
You must be signed in to change notification settings - Fork 202
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
Allow default improver to improve without affected packages #693
Conversation
f89dc23
to
c5deeb1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM... but consider using file-driven tests for easier maintenance and updates. You my the nginx test branch merged first.
1576ec9
to
bfaa17a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
b346581
to
7b337cd
Compare
Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
7b337cd
to
bc94910
Compare
""" | ||
default_improver = DefaultImprover() | ||
response_file = os.path.join(TEST_DATA, f"alpine-input.json") | ||
with open(response_file, "r") as f: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"r" is always the default FWIW, so not needed in general
Signed-off-by: Tushar Goel tushar.goel.dav@gmail.com