Skip to content

Release zip 2.1.0 #61

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

Closed
12 tasks done
gaborcsardi opened this issue Jul 15, 2020 · 7 comments
Closed
12 tasks done

Release zip 2.1.0 #61

gaborcsardi opened this issue Jul 15, 2020 · 7 comments

Comments

@gaborcsardi
Copy link
Member

gaborcsardi commented Jul 15, 2020

Prepare for release:

  • devtools::check_win_devel() seems dead
  • rhub::check_for_cran()
  • rhub::check(platform = "solaris-x86-patched")
  • rhub::check(platform = "ubuntu-rchk")
  • revdepcheck::revdep_check(num_workers = 4)
  • Polish NEWS
  • If new failures, update email.yml then revdepcheck::revdep_email_maintainers()

Perform release:

  • Bump version (in DESCRIPTION and NEWS)
  • devtools::submit_cran()
  • Approve email

Wait for CRAN...

  • Tag release
  • Bump dev version

Template from r-lib/usethis#338

@gaborcsardi
Copy link
Member Author

@davidgohel This should not break anything (although I haven't run the revdep checks yet), but I had to rewrite a lot of code for Unicode support so you might want to test officer with this. Thanks!

@davidgohel
Copy link

thanks - I did a quick test on a colleague Windows machine, so far everything looks ok.
I will do more tests tomorrow and let you know if something broke

@davidgohel
Copy link

Funny, I did that fix yesterday: davidgohel/officer@8c7926b

From your code, I understand I should have used first enc2utf8, then Encoding(blah) <- "UTF-8"? (ignore my question if no time...)

@gaborcsardi
Copy link
Member Author

gaborcsardi commented Jul 15, 2020

It depends what target is in your code. E.g. if it is a path from the zip file, then it is already UTF-8. Well, it is in the new version, if the zip file was created by zip.

@davidgohel
Copy link

@gaborcsardi OK, thanks for having worked on that

@davidgohel
Copy link

Hi,

A quick confirmation, the new version is working well and is solving issues with Windows paths or filename containing non ascii characters

@gaborcsardi
Copy link
Member Author

@davidgohel Great, thanks!

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

No branches or pull requests

2 participants