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

Gas price simulation data retriever #2533

Merged
merged 18 commits into from
Jan 8, 2025
Merged

Conversation

acerone85
Copy link
Contributor

@acerone85 acerone85 commented Jan 6, 2025

Linked Issues/PRs

Description

  • Adds a data retrieval service that fetches L2 and L1 cost data for gas price v1 simulations

Checklist

  • Breaking changes are clearly marked as such in the PR description and changelog
  • New behavior is reflected in tests
  • The specification matches the implemented behavior (link update PR if changes are needed)

Before requesting review

  • I have reviewed the code myself
  • I have created follow-up issues caused by this PR and linked them here

After merging, notify other teams

[Add or remove entries as needed]

@rymnc
Copy link
Member

rymnc commented Jan 6, 2025

can you please fix it according to the broken ci :)

@acerone85 acerone85 added the no changelog Skip the CI check of the changelog modification label Jan 6, 2025
@acerone85
Copy link
Contributor Author

can you please fix it according to the broken ci :)

done :)

## Linked Issues/PRs
<!-- List of related issues/PRs -->
- #2533 

## Description
<!-- List of detailed changes -->

## Checklist
- [ ] Breaking changes are clearly marked as such in the PR description
and changelog
- [ ] New behavior is reflected in tests
- [ ] [The specification](https://github.com/FuelLabs/fuel-specs/)
matches the implemented behavior (link update PR if changes are needed)

### Before requesting review
- [ ] I have reviewed the code myself
- [ ] I have created follow-up issues caused by this PR and linked them
here

### After merging, notify other teams

[Add or remove entries as needed]

- [ ] [Rust SDK](https://github.com/FuelLabs/fuels-rs/)
- [ ] [Sway compiler](https://github.com/FuelLabs/sway/)
- [ ] [Platform
documentation](https://github.com/FuelLabs/devrel-requests/issues/new?assignees=&labels=new+request&projects=&template=NEW-REQUEST.yml&title=%5BRequest%5D%3A+)
(for out-of-organization contributors, the person merging the PR will do
this)
- [ ] Someone else?
@rymnc rymnc enabled auto-merge (squash) January 8, 2025 17:02
@rymnc rymnc merged commit 5fad38a into master Jan 8, 2025
31 checks passed
@rymnc rymnc deleted the gas_price_simulation_data_retriever branch January 8, 2025 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog Skip the CI check of the changelog modification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants