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

Run xref on Travis CI #1317

Merged
merged 5 commits into from Jun 21, 2016
Merged

Run xref on Travis CI #1317

merged 5 commits into from Jun 21, 2016

Conversation

ddeboer
Copy link
Member

@ddeboer ddeboer commented Jun 21, 2016

Fix #1316.

@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @mworrell and @arjan to be potential reviewers

@ddeboer
Copy link
Member Author

ddeboer commented Jun 21, 2016

@mworrell Please fix the partial mod_survey merge to solve the missing function call. 😉

* Remove erlydtl test
* Fix m_site
* Update z_stdlib
@mworrell
Copy link
Member

That scomp can be removed.

@mworrell
Copy link
Member

We now have the following error:

===> Warning: erlydtl_parser:return_error/2 is unused local function (Xref)

This is inside the generated Yecc parser.
Either we find a way to disable checks for certain modules or merge this after merging the new template compiler.

@ddeboer
Copy link
Member Author

ddeboer commented Jun 21, 2016

===> Warning: erlydtl_parser:return_error/2 is unused local function (Xref)`

@mworrell This one is fixed, some new ones popped up after merging in the new template compiler. Feel like fixing those? 😉

@mworrell mworrell merged commit fc846ad into master Jun 21, 2016
@mworrell mworrell deleted the xref-travis branch June 21, 2016 15:22
@mworrell
Copy link
Member

Merged, good work @ddeboer

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

Successfully merging this pull request may close these issues.

None yet

3 participants