Skip to content

Conversation

xunilrj
Copy link
Contributor

@xunilrj xunilrj commented Sep 1, 2025

Description

This PR brings all the scripts that we have, and push a couple more into one just file (https://github.com/casey/just).

Initially we are just calling the script directly, but we can improve it later.

Checklist

  • I have linked to any relevant issues.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have updated the documentation where relevant (API docs, the reference, and the Sway book).
  • I have added tests that prove my fix is effective or that my feature works.
  • I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
  • I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
  • I have requested a review from the relevant team or maintainers.

@xunilrj xunilrj marked this pull request as ready for review September 1, 2025 14:17
@xunilrj xunilrj requested review from a team as code owners September 1, 2025 14:17
JoshuaBatty
JoshuaBatty previously approved these changes Sep 2, 2025
Copy link
Member

@JoshuaBatty JoshuaBatty left a comment

Choose a reason for hiding this comment

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

nice!

@JoshuaBatty JoshuaBatty requested review from a team September 2, 2025 00:30
zees-dev
zees-dev previously approved these changes Sep 2, 2025
Copy link
Contributor

@zees-dev zees-dev left a comment

Choose a reason for hiding this comment

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

Nice; LGTM 👍

P.S. Could probably specify in readme/doc(s) to use just for running commands

@xunilrj
Copy link
Contributor Author

xunilrj commented Sep 2, 2025

P.S. Could probably specify in readme/doc(s) to use just for running commands

Good idea! Done.

@xunilrj xunilrj enabled auto-merge (squash) September 3, 2025 13:24
@xunilrj xunilrj merged commit 567b5b8 into master Sep 4, 2025
39 checks passed
@xunilrj xunilrj deleted the xunilrj/justfile branch September 4, 2025 00:05
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.

4 participants