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

Scary cleaning V2 #24

Open
wants to merge 431 commits into
base: cleaned
Choose a base branch
from

Conversation

odelmarcelle
Copy link
Contributor

@odelmarcelle odelmarcelle commented Apr 10, 2021

The result of a cleaning with BFG repo cleaner.

The cleaning correct a large number of commits in the past.

Former-commit-id: e3bd55d
Former-commit-id: 35d10d2
Former-commit-id: 168f589
Former-commit-id: 7c933f9
Former-commit-id: 7954935
Former-commit-id: 8d04d0d
Former-commit-id: 698ee41
Former-commit-id: 412b3b4
Former-commit-id: c5dc392
Former-commit-id: 9b85ac8
Former-commit-id: 7acca8d
Former-commit-id: cee91a4
Former-commit-id: 4216462
Former-commit-id: 5fa2325
Former-commit-id: c54bb03
Former-commit-id: 527a677
Former-commit-id: e03fda8
Former-commit-id: 6645321
Former-commit-id: 88ba057
sborms and others added 26 commits June 26, 2020 13:19
Former-commit-id: 8f4ad6d
Former-commit-id: 6839f76
Former-commit-id: d31ff6b
Former-commit-id: 1964c21
Former-commit-id: e809daa
Former-commit-id: e32d82d
Former-commit-id: 7e1070a
Former-commit-id: 20a6eef
Former-commit-id: 77f6ff7
Former-commit-id: abd4bbd
Former-commit-id: d9b8495
Former-commit-id: 80848f7
Former-commit-id: 3d19698
Former-commit-id: fb28aba
Former-commit-id: 4886b86
Former-commit-id: a90680e
Former-commit-id: c9485a8
Former-commit-id: a84b124
Former-commit-id: 41a4806
- texts() is replaced by as.character()


Former-commit-id: 00df984
Change usage to avoid warnings for quanteda v3

Former-commit-id: 73b067b
@sborms
Copy link
Collaborator

sborms commented Apr 10, 2021

That looks indeed very scary!

Can you push it as a separate branch? Then I can investigate better if there are unintended consequences, before we merge it into master.

@odelmarcelle odelmarcelle changed the base branch from master to cleaned April 12, 2021 01:23
@sborms
Copy link
Collaborator

sborms commented Apr 26, 2021

Let me put it on hold for a while. Will for sure come back to this (probably after official publication in JSS), because it is a good idea.

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.

6 participants