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 forc-index plugin #372

Merged
merged 3 commits into from
Nov 30, 2022
Merged

Add forc-index plugin #372

merged 3 commits into from
Nov 30, 2022

Conversation

ra0x3
Copy link
Contributor

@ra0x3 ra0x3 commented Nov 29, 2022 •
edited
Loading

Description

  • This PR adds the long-awaited Forc Indexer plugin that is invoked via forc index
  • forc index is pretty much based on forc in terms of structure
  • There may have been a few things that I left out (it's been a minute since I've committed to forc)
  • I'm not really sure how to "hook this in" to forc -- so I just built the binary cargo build --release -p forc-index and manually added it to my ~/.fuelup/bin

Testings steps

  • I would recommend you pull down the https://github.com/FuelLabs/fuel-indexer/ project
  • And build a fuel-indexer binary via cargo build --release -p fuel-indexer
    • This will help with the forc index start and forc index deploy commands
    • This fuel-indexer binary has been merged into fuelup but haven't had a chance to use it yet
  • Then maybe just start playing around by following the forc index README

@ra0x3 ra0x3 self-assigned this Nov 29, 2022
@ra0x3 ra0x3 marked this pull request as draft November 29, 2022 17:37
@ra0x3 ra0x3 linked an issue Nov 29, 2022 that may be closed by this pull request
Create a forc-indexer plugin #294
Closed
@ra0x3 ra0x3 requested a review from deekerno November 29, 2022 19:37
@ra0x3 ra0x3 marked this pull request as ready for review November 29, 2022 19:38
@ra0x3 ra0x3 mentioned this pull request Nov 30, 2022
Merged
kayagokalp
kayagokalp approved these changes Nov 30, 2022
View reviewed changes
Copy link
Member

@kayagokalp kayagokalp left a comment

Choose a reason for hiding this comment

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

looks really 🆒

@ra0x3 ra0x3 merged commit 820f343 into master Nov 30, 2022
@ra0x3 ra0x3 deleted the rashad/294-forc-index-plugin branch November 30, 2022 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@deekerno deekerno deekerno approved these changes

@kayagokalp kayagokalp kayagokalp 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.

Create a forc-indexer plugin
3 participants
@ra0x3 @deekerno @kayagokalp