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.

Sign up for GitHub

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

Jump to bottom

Feat/18 packagize #21

Closed
wants to merge 32 commits into from
Closed

Feat/18 packagize #21

wants to merge 32 commits into from

Conversation

AlexAxthelm
Copy link
Contributor

@AlexAxthelm AlexAxthelm commented Nov 22, 2024 •
edited
Loading

@AlexAxthelm
update docker dirs and gitignore
6397fda
@AlexAxthelm
Move existing functions to own files
b8bc248
@AlexAxthelm
Wrap main script as a function
a518ce2
@AlexAxthelm
Move file to R/
11bfca8
@AlexAxthelm
Use script in inst directory, and install package
a0dc1c9
@AlexAxthelm
update main calling script
a5189e1
@AlexAxthelm
Merge branch 'main' into feat/18-packagize
bbf3830
@AlexAxthelm
Move new functions to own files
0f8ad76
@AlexAxthelm
Add R package check workflow
f3cadd5
@AlexAxthelm
Add directories as args to main function
de661f5
@AlexAxthelm
Add .lintr with all errors suppressed
659f87a
@AlexAxthelm
Linting: indentation
239228f
@AlexAxthelm
Style: use explicit pivoting columns
c3b2a06 
(include, rather than exclude)
@AlexAxthelm
linting (trailing whitespace)
efb6201
@AlexAxthelm
linting: line length
81e9ddb
@AlexAxthelm
Linting: assignment (=/<-) and commas
73c6fe1
@AlexAxthelm
Prepare for object_usage_linter
dfa2d99
@AlexAxthelm
Linting: First pass object_usage
39fc9a7
@AlexAxthelm
import functions from logger
cb888f3
@AlexAxthelm
namespace calls to filter
70ce560
@AlexAxthelm
namespace calls to mutate
11cf740
@AlexAxthelm
namespace calls to group_by/ungroup
e14b97b
@AlexAxthelm
namespace functions form tidyr
3b295b0
@AlexAxthelm
remove most of the library calls
2a96bf0
@AlexAxthelm
Convert from %>% pipe to |>
58641ed
@AlexAxthelm
fix join syntax and namespace dplyr::if_else
2ee4840
@AlexAxthelm
ignore output dir
aeacffd
@AlexAxthelm
namespace remaining dplyr functions
bdacd04
@codecov Codecov
Copy link

codecov bot commented Nov 25, 2024

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@AlexAxthelm AlexAxthelm mentioned this pull request Nov 25, 2024
Closed
@AlexAxthelm AlexAxthelm mentioned this pull request Dec 2, 2024
Merged
@AlexAxthelm
Copy link
Contributor Author

closing in favor of #30

@AlexAxthelm AlexAxthelm closed this Dec 3, 2024
@AlexAxthelm AlexAxthelm deleted the feat/18-packagize branch December 6, 2024 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@AlexAxthelm