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

33 use standard NEST argument convention documentation #46

Merged
merged 25 commits into from
Sep 23, 2021
Merged

33 use standard NEST argument convention documentation #46

merged 25 commits into from
Sep 23, 2021

Conversation

BFalquet
Copy link
Contributor

close #33

adjust argument type. Thank you for the review.

benoit and others added 23 commits September 9, 2021 17:37
initial creation ext01
5296ca0
Merge branch 'main' into 18_ext01@main
33768b5
Add overall col
66acff7
Add possibility to select parameters
3e9ff80
remove unnecessary step
3aac3d6 
(transformation as factor: maybe needed later)
Add magrittr and document
1dfc5a3
Linting and vectorization
eb271b7
Merge branch 'main' into 18_ext01@main
0fc9923
export layouts
93bc7af
work in progress
b93c2e5
New utility function cut_by_group used in examples
a18b979
Introduce assert_colnames function
959e975
correct arg to pass checks
3fa8414
Merge branch 'main' into 34_export-layouts@main
ae25536
Rationalize documentation
3f126a5
Export layout and clean args
d3af43a
@BFalquet
Merge branch 'main' into 18_ext01@main
5617fe4
Correction to pass checks
b0433b3
Merge remote-tracking branch 'origin/18_ext01@main' into 34_export-la…
ac09c5e 
…youts@main
Update pkgdown
6eca0e2
Merge branch '34_export-layouts@main' into 33_arg_convention@main
3d3dbf7
adjust arg type in functions
c446650
adjust arg type in gen_args
4735522
@waddella waddella self-assigned this Sep 23, 2021
@BFalquet BFalquet merged commit 968d3b8 into main Sep 23, 2021
@BFalquet BFalquet deleted the 33_arg_convention@main branch September 23, 2021 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@waddella waddella waddella approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

use standard NEST argument convention documentation
2 participants
@BFalquet @waddella