Skip to content
This repository has been archived by the owner on Oct 25, 2024. It is now read-only.

chore: update deployment start #1502

Merged
merged 2 commits into from
Dec 6, 2023
Merged

chore: update deployment start #1502

merged 2 commits into from
Dec 6, 2023

Conversation

ra0x3
Copy link
Contributor

@ra0x3 ra0x3 commented Dec 6, 2023 •
edited
Loading

Thanks for opening a PR with the Fuel Indexer project. Please review the Checklist below and ensure you've completed all of the necessary steps to make this PR review as painless as possible.

Checklist

  • Ensure your top-level commit message is in line with our contributor guidelines.
  • Please add proper labels.
  • If there is an issue associated with this PR, please link the issue (right-hand sidebar)
  • If there is not an issue associated with this PR, add this PR to the "Fuel Indexer" project (right-hand sidebar)
  • Please allow Codeowners at least 24 hours to do a first-pass review.
  • Please add thoroughly detailed testing steps below.
  • Please keep your Changelog message short and sweet.

Description

  • Updates our deployment start to delay a bit (so as to not trigger rate limiting)
  • Also bumps the block page size

Testing steps

  • QA Results attached below

Changelog

  • chore: update deployment start

@ra0x3 ra0x3 self-assigned this Dec 6, 2023
@ra0x3 ra0x3 force-pushed the rashad/update-deployment-start branch from 4c1048f to 68a8fee Compare December 6, 2023 13:37
@ra0x3
Copy link
Contributor Author

ra0x3 commented Dec 6, 2023

QA Results

Rashads-MacBook-Air.local-2023-12-06.indexer-qa.txt

@ra0x3 ra0x3 marked this pull request as ready for review December 6, 2023 15:07
lostman
lostman approved these changes Dec 6, 2023
View reviewed changes
Copy link
Contributor

@lostman lostman left a comment

Choose a reason for hiding this comment

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

🛳️

deekerno
deekerno approved these changes Dec 6, 2023
View reviewed changes
Copy link
Contributor

@deekerno deekerno left a comment

Choose a reason for hiding this comment

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

🚄

@ra0x3 ra0x3 merged commit 8dc3654 into develop Dec 6, 2023
19 checks passed
@ra0x3 ra0x3 deleted the rashad/update-deployment-start branch December 6, 2023 15:10
@ra0x3 ra0x3 mentioned this pull request Dec 6, 2023
Merged
7 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@deekerno deekerno deekerno approved these changes

@lostman lostman lostman approved these changes

Assignees

@ra0x3 ra0x3

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ra0x3 @lostman @deekerno