Skip to content
This repository has been archived by the owner on May 12, 2018. It is now read-only.

Allow behaviors defined inside xref_extra_path #530

Merged
merged 1 commit into from
Aug 21, 2015
Merged

Allow behaviors defined inside xref_extra_path #530

merged 1 commit into from
Aug 21, 2015

Conversation

paulo-ferraz-oliveira
Copy link

Use case: xref'ing a module that defined behavior ranch_protocol was
exiting with
{'EXIT',{undef,[{ranch_protocol,behaviour_info, [callbacks],[]}

@ferd
Copy link
Contributor

ferd commented Aug 14, 2015

I'm okay with this change. Could we have a test to go with it? See https://github.com/rebar/rebar/blob/master/CONTRIBUTING.md

@paulo-ferraz-oliveira
Copy link
Author

Thanks for your feedback, Fred. I'll try to have a test case ready soon (not this weekend though) that illustrates the desired behavior (should I have one for success and another for non-success?) and then ammend my commit.

@ferd
Copy link
Contributor

ferd commented Aug 14, 2015

Success alone should be good enough.

@paulo-ferraz-oliveira
Copy link
Author

I updated the pull request. Created only one test case for retest, which illustrates the expected behavior.

Use case: xref'ing a module that defined behavior ranch_protocol was
exiting with
{'EXIT',{undef,[{ranch_protocol,behaviour_info, [callbacks],[]}
ferd added a commit that referenced this pull request Aug 21, 2015
…behaviors

Allow behaviors defined inside xref_extra_path
@ferd ferd merged commit a197b29 into rebar:master Aug 21, 2015
@paulo-ferraz-oliveira paulo-ferraz-oliveira deleted the fix/code_path_for_behaviors branch February 10, 2018 02:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants