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

Fuel Core v0.16.1 #813

Merged
merged 10 commits into from
Feb 2, 2023
Merged

Fuel Core v0.16.1 #813

merged 10 commits into from
Feb 2, 2023

Conversation

Voxelot
Copy link
Member

@Voxelot Voxelot commented Feb 1, 2023

Description of changes

Update fuels-rs to the latest fuel-core 0.16.1 and latest sway 0.34.0.

Due to changes in fuel-asm for GTF, this is causing yet another circular upgrade issue.

  • Used { workspace = true } syntax to use one version across the project and simplify integration with new changes in the future.
  • Added ci_checks.sh script to compile all tests to simplify integration of changes like this in the future.
  • Update contracts to be compatible with sway 0.34.0.
  • Updated tests to be compatible with fuel-core 0.16.1.

related PR: FuelLabs/sway#3961

xgreenx and others added 3 commits February 1, 2023 17:30
Fixed all tests.
Added `ci_checks.sh` that perform almost all CI checks locally by one command.
@xgreenx xgreenx marked this pull request as ready for review February 1, 2023 18:18
@digorithm digorithm requested review from a team February 1, 2023 18:45
digorithm
digorithm previously approved these changes Feb 1, 2023
Copy link
Member

@digorithm digorithm left a comment

Choose a reason for hiding this comment

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

Thanks @Voxelot and @xgreenx! Adding fuel-* to [workspace.dependencies] is super handy; not sure why we hadn't done this before.

@Voxelot
Copy link
Member Author

Voxelot commented Feb 1, 2023

Thanks @Voxelot and @xgreenx! Adding fuel-* to [workspace.dependencies] is super handy; not sure why we hadn't done this before.

The crate publishing action didn't support it until Green refactored it to use cargo metadata instead of ad-hoc toml parsing in ts. katyo/publish-crates#600

@Voxelot Voxelot merged commit fa7beed into master Feb 2, 2023
@Voxelot Voxelot deleted the Voxelot/fuel-core-0.16 branch February 2, 2023 07:19
@digorithm digorithm restored the Voxelot/fuel-core-0.16 branch February 2, 2023 21:46
@hal3e hal3e deleted the Voxelot/fuel-core-0.16 branch February 8, 2023 22:14
MujkicA pushed a commit that referenced this pull request Feb 10, 2023
Update `fuels-rs` to the latest `fuel-core 0.16.1` and latest `sway
0.34.0`.

Due to changes in `fuel-asm` for GTF, this is causing yet another
circular upgrade issue.
- Used `{ workspace = true }` syntax to use one version across the
project and simplify integration with new changes in the future.
- Added `ci_checks.sh` script to compile all tests to simplify
integration of changes like this in the future.
- Update contracts to be compatible with `sway 0.34.0`.
- Updated tests to be compatible with `fuel-core 0.16.1`.

related PR: FuelLabs/sway#3961

---------

Co-authored-by: green <xgreenx9999@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants