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.

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

(3/7) RFC79: Implement incremental upload for timeline data #43

Merged
merged 1 commit into from
Jun 19, 2024

Conversation

forus
Copy link
Contributor

@forus forus commented Jun 4, 2024

Please review #42 next.

Please note, to avoid the costly process of rebasing all pull requests (PRs), all feedback will be incorporated into the final PR.

Copy link
Member

@pieterlukasse pieterlukasse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good.
Two minor suggestions.

@forus forus requested a review from pieterlukasse June 11, 2024 19:14
Copy link
Collaborator

@haynescd haynescd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Base automatically changed from make_study_es_0_to_load to rfc79 June 19, 2024 14:21
@forus forus merged commit a5ac232 into rfc79 Jun 19, 2024
4 checks passed
@forus forus deleted the inc-timeline-uploader branch June 19, 2024 14:22
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.

3 participants