prevent Unknown resource type error (issue #35) - #36
Conversation
|
Hi @ttoohey, thanks for this patch. Are you able to provide a unit test to prove that this now compiles? |
|
It looks like #10 was completely broken when merged. Looks like a rebase or merge from master went wrong and undid part of the changes. |
|
I think the issue was #7 and #10 both were trying to fix the same issue. This PR undoes part of #10, but also leaves parts of it behind. Maybe we just revert #10 completely? |
alexjfisher
left a comment
There was a problem hiding this comment.
I think I'd prefer reverting #10 ?
@bastelfreak ?
|
@bastelfreak - I'd like to have included a unit test, but I couldn't get past an error when trying to @alexjfisher - yes, #37 solves my problem. I'm not sure it will solve what #10 was attempting (but this PR probably doesn't either). |
|
Dear @ttoohey, thanks for the PR! This is pccibot, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase? You can find my sourcecode at voxpupuli/vox-pupuli-tasks |
|
Dear @ttoohey, thanks for the PR! This is pccibot, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase? You can find my sourcecode at voxpupuli/vox-pupuli-tasks |
No description provided.