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

Replace chunks with qenv #129

Closed
asbates opened this issue Apr 14, 2023 · 1 comment
Closed

Replace chunks with qenv #129

asbates opened this issue Apr 14, 2023 · 1 comment
Labels
core

Comments

@asbates
Copy link
Contributor

asbates commented Apr 14, 2023

There may be more places but data_merge_srv at least still uses chunks. This should be replaced with qenv.
@asbates asbates mentioned this issue Apr 14, 2023
Merged
@donyunardi donyunardi mentioned this issue Apr 14, 2023
Closed
@donyunardi
Copy link
Contributor

Acceptance Criteria

  • Replace and remove chunk in data_merge_srv
  • Update the roxygen example of merge_expression_module to not use chunk

@donyunardi donyunardi mentioned this issue Apr 14, 2023
Closed
asbates added a commit that referenced this issue Apr 17, 2023
@asbates @dependabot-preview
Update docs (#130)
801b461 
Updates some documentation in preparation for release. These changes
regard examples in the roxygen docs.

There are still a few more documentation changes to be made. However
they are beyond what I can do quickly so I opened #128.

Also note there is still some chunks usage. I've opened #129 to address
this.

Fixes #125

---------

Co-authored-by: 27856297+dependabot-preview[bot]@users.noreply.github.com <27856297+dependabot-preview[bot]@users.noreply.github.com>
@gogonzo gogonzo closed this as completed Aug 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
core
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@gogonzo @donyunardi @asbates