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

fix unknown variable: staged_package #179

Merged
merged 2 commits into from
Apr 3, 2018

Conversation

bastelfreak
Copy link
Member

No description provided.

@bastelfreak bastelfreak added the bug Something isn't working label Mar 28, 2018
@bastelfreak
Copy link
Member Author

This is a proposed fix for #169. This reveals new errors that I don't quite understand yet. I'm not sure if we've multiple bugs in the code or if this patch is wrong. @TraGicCode and idea?

@TraGicCode
Copy link
Contributor

Yea i believe that was what was proposed. It's more of a patch then a true fix of the actual problem.

@TraGicCode
Copy link
Contributor

This seems fine to me. I think the these errors were hidden by this bug you just fixed. Can i merge this?

Copy link
Contributor

@TraGicCode TraGicCode left a comment

Choose a reason for hiding this comment

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

LGTM 👍 . Doesn't actually solve the underlying mess but does fix this issue :).

Copy link
Contributor

@TraGicCode TraGicCode left a comment

Choose a reason for hiding this comment

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

Per my conversation with bastelfreak before we merge this lets fix the next unit test failures that cropped up.

@michaelweiser michaelweiser mentioned this pull request Mar 28, 2018
@TraGicCode
Copy link
Contributor

Based on the changes #180 was trying to make i understand now why it's failing. I reported this a while back #151.

@TraGicCode TraGicCode dismissed their stale review April 3, 2018 11:47

I've started contributing to help fix the issue.

@bastelfreak bastelfreak merged commit 1568a19 into voxpupuli:master Apr 3, 2018
@bastelfreak bastelfreak deleted the filepath_error branch April 3, 2018 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants