Skip to content

Conversation

@mmaguero
Copy link

@mmaguero mmaguero commented Jul 17, 2019

Objective:

Hi!
I hope you can add this feature to the project.

@coveralls
Copy link

coveralls commented Jul 17, 2019

Coverage Status

Coverage remained the same at 96.471% when pulling a30a1c0 on mmaguero:master into 07657a2 on fnielsen:master.

@AndreiRegiani
Copy link

What can I do to help to get this merged?

@fnielsen
Copy link
Owner

@AndreiRegiani Thanks for the PR. Sorry for my delay. Two issues: I am wondering whether the license for the Spanish word list at https://github.com/jboscomendoza/rpubs/tree/master/sentimientos_afinn It is not immediately apparent to me. The other issue is the name of the data file. Is the file from tidytext?

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.

4 participants