Skip to content

Improve error message for overpacked tiles #126

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
Merged
merged 1 commit into from
Jun 4, 2025
Merged

Improve error message for overpacked tiles #126

merged 1 commit into from
Jun 4, 2025

Conversation

jansel
Copy link
Contributor

@jansel jansel commented Jun 3, 2025 •
edited
Loading

jansel added a commit that referenced this pull request Jun 3, 2025
@jansel
Improve error message for overpacked tiles
77cf147 
stack-info: PR: #126, branch: jansel/stack/22
@jansel jansel force-pushed the jansel/stack/22 branch from 836e85d to 77cf147 Compare June 3, 2025 22:44
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label Jun 3, 2025
This was referenced Jun 3, 2025
Merged
Merged
@jansel jansel changed the base branch from jansel/stack/21 to main June 3, 2025 22:47
jansel added a commit that referenced this pull request Jun 3, 2025
@jansel
Improve error message for overpacked tiles
55d2c85 
stack-info: PR: #126, branch: jansel/stack/22
@jansel jansel force-pushed the jansel/stack/22 branch from 77cf147 to 55d2c85 Compare June 3, 2025 22:47
@jansel jansel changed the base branch from main to jansel/stack/21 June 3, 2025 22:47
@jansel jansel changed the base branch from jansel/stack/21 to main June 3, 2025 22:51
jansel added a commit that referenced this pull request Jun 3, 2025
@jansel
Improve error message for overpacked tiles
27e9a69 
stack-info: PR: #126, branch: jansel/stack/22
@jansel jansel force-pushed the jansel/stack/22 branch from 55d2c85 to 27e9a69 Compare June 3, 2025 22:51
@jansel jansel changed the base branch from main to jansel/stack/21 June 3, 2025 22:51
@jansel jansel changed the base branch from jansel/stack/21 to main June 4, 2025 01:01
@jansel jansel changed the base branch from main to jansel/stack/21 June 4, 2025 01:01
@jansel jansel mentioned this pull request Jun 4, 2025
Merged
@jansel jansel changed the base branch from jansel/stack/21 to main June 4, 2025 01:02
@jansel jansel changed the base branch from main to jansel/stack/21 June 4, 2025 01:02
@jansel jansel changed the base branch from jansel/stack/21 to main June 4, 2025 01:12
@jansel jansel changed the base branch from main to jansel/stack/21 June 4, 2025 01:12
@jansel jansel requested a review from drisspg June 4, 2025 01:58
@jansel jansel changed the base branch from jansel/stack/21 to main June 4, 2025 02:23
@jansel jansel changed the base branch from main to jansel/stack/21 June 4, 2025 02:23
This was referenced Jun 4, 2025
Merged
Merged
jansel added a commit that referenced this pull request Jun 4, 2025
@jansel
Improve error message for overpacked tiles
6437eeb 
stack-info: PR: #126, branch: jansel/stack/22
@jansel jansel force-pushed the jansel/stack/22 branch from 27e9a69 to 6437eeb Compare June 4, 2025 02:24
@jansel jansel changed the base branch from jansel/stack/21 to main June 4, 2025 02:25
@jansel jansel force-pushed the jansel/stack/22 branch from 6437eeb to 4a5e3dd Compare June 4, 2025 02:30
yf225
yf225 reviewed Jun 4, 2025
View reviewed changes
This was referenced Jun 4, 2025
Merged
Merged
Merged
@jansel
Improve error message for overpacked tiles
d8f53ea 
stack-info: PR: #126, branch: jansel/stack/22
@jansel jansel force-pushed the jansel/stack/22 branch from 4a5e3dd to d8f53ea Compare June 4, 2025 04:55
@jansel jansel merged commit c145af0 into main Jun 4, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yf225 yf225 yf225 approved these changes

@drisspg drisspg Awaiting requested review from drisspg

Assignees
No one assigned
Labels
CLA Signed This label is managed by the Meta Open Source bot.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jansel @yf225 @facebook-github-bot