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

Equip with Spanish #24

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Equip with Spanish #24

wants to merge 5 commits into from

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.

None yet

4 participants