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

Add padding details #95

Merged
merged 1 commit into from
Sep 25, 2023
Merged

Add padding details #95

merged 1 commit into from
Sep 25, 2023

Conversation

alexmturner
Copy link
Collaborator

@alexmturner alexmturner commented Sep 12, 2023 •
edited
Loading

Updates the explainer with the short-term and longer-term plans for padding.

See #56 for more discussion and #98 for the spec change.

@alexmturner
Add padding details
2f314b5 
Updates the explainer with the short-term and longer-term plans for padding.
@alexmturner
Copy link
Collaborator Author

@menonasha @linnan-github, could you PTAL? Thanks!

@alexmturner alexmturner mentioned this pull request Sep 21, 2023
Merged
@alexmturner alexmturner merged commit ce92db6 into main Sep 25, 2023
@alexmturner alexmturner deleted the alexmturner-padding-plan branch September 25, 2023 21:56
alexmturner added a commit that referenced this pull request Sep 26, 2023
@alexmturner
Spec: Pad the payload with null contributions (#98)
813a62d 
Ensures that the payload always has a fixed number of contributions by
adding (0,0) contributions.

See #56 for more discussion and #95 for the corresponding spec change.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@menonasha menonasha menonasha 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.
2 participants
@alexmturner @menonasha