Skip to content

Conversation

@PaulLerner
Copy link
Contributor

cc @yjernite is it ok like this?

@lhoestq lhoestq requested a review from yjernite June 10, 2021 08:38
@yjernite
Copy link
Member

LGTM, but I'm not sure what's going on with the Unix tests @lhoestq

@lhoestq
Copy link
Member

lhoestq commented Jun 14, 2021

The CI error is unrelated to this PR, it's been fixed now on master.

@yjernite yjernite merged commit 236c7fa into huggingface:master Jun 14, 2021
@yjernite
Copy link
Member

Thanks @PaulLerner !

@aDrIaNo34500
Copy link

#- [ ] - Heyimage@fr[fr_fr**fr~~fr `fr```
FR

@lhoestq
Copy link
Member

lhoestq commented Jun 15, 2021

Oh that was unexpected. I didn't know pokemons were into NLP

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