Skip to content

Miscellaneous small package cleanups#485

Merged
eddelbuettel merged 4 commits intomasterfrom
de/sc-23559/mixed_cleanups
Nov 17, 2022
Merged

Miscellaneous small package cleanups#485
eddelbuettel merged 4 commits intomasterfrom
de/sc-23559/mixed_cleanups

Conversation

@eddelbuettel
Copy link
Copy Markdown
Contributor

This PR cleans up three distinct corners none of which affect the key functionality

  • rename two R files to uppercase matching all other files in R/
  • rename the codecov.yml file to lead with a dot which is also supported and 'hides' it a little
  • skips some final array tests if in older versions 2.4.* and 2.10.*, this came up in other CI tests

@shortcut-integration
Copy link
Copy Markdown

This pull request has been linked to Shortcut Story #23559: Miscellaneous small package improvements.

While there clearly are repos on GitHub using .codecov.yml a glance at the
covr code as well as the codecov.io documentation does not suggest those are
actually being read.  :-/
@eddelbuettel eddelbuettel merged commit 036a785 into master Nov 17, 2022
@eddelbuettel eddelbuettel deleted the de/sc-23559/mixed_cleanups branch November 17, 2022 21:38
eddelbuettel added a commit that referenced this pull request Nov 18, 2022
eddelbuettel added a commit that referenced this pull request Nov 24, 2022
eddelbuettel added a commit that referenced this pull request Nov 28, 2022
eddelbuettel added a commit that referenced this pull request Nov 29, 2022
* Default sparse read via tiledb_array to UNORDERED

* Use nchar(layout) not length, and wrap in isTRUE for comparison

* Update NEWS.md for #483, #484, #485, and #487

* Update NEWS for #488 [ci skip]

* Roll micro release to 0.16.0.3 [ci skip]

* Add codecov.yml to .Rbuildignore

* Tighten one version-based test exclusion
@eddelbuettel eddelbuettel mentioned this pull request Dec 1, 2022
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.

2 participants