Skip to content

Conversation

@jaoleal
Copy link
Collaborator

@jaoleal jaoleal commented Oct 16, 2025

What is the purpose of this pull request?

  • Bug fix
  • Documentation update
  • New feature
  • Test
  • Other:

Which crates are being modified?

  • floresta-chain
  • floresta-common
  • floresta-compact-filters
  • floresta-electrum
  • floresta-node
  • floresta-rpc
  • floresta-watch-only
  • floresta-wire
  • bin/florestad
  • bin/floresta-cli
  • Other: justfile, readme and docs/readme

Description and Notes

Part of #661

This proposes a set of justfile groups as well documentation for them.

What yall think of adding linting for justfile into CI ?

How to verify the changes you have done?

I created a new docs/justfile.md that explains the groups and what they mean so,

read docs/justfile.md and judge wheter they are concise and meaningfull.

Theres new docs about justfile in CONTRIBUTING , README and docs/README. These should be instructions for a contributor or a developer to understand the scripts we have.

Contributor Checklist

  • I've followed the contribution guidelines
  • I've verified one of the following:
    • Ran just pcc (recommended but slower)
    • Ran just lint-features '-- -D warnings' && cargo test --release
    • Confirmed CI passed on my fork
  • I've linked any related issue(s) in the sections above

Finally, you are encouraged to sign all your commits (it proves authorship and guards against tampering—see How (and why) to sign Git commits and GitHub's guide to signing commits).

@jaoleal
Copy link
Collaborator Author

jaoleal commented Oct 16, 2025

Sorry, there was a typo.

CC @JoseSK999.

The groups i mentioned in #661

@Davidson-Souza Davidson-Souza requested a review from qlrd October 17, 2025 11:07
@Davidson-Souza Davidson-Souza added enhancement New feature or request code quality Generally improves code readability and maintainability labels Oct 17, 2025
@joaozinhom
Copy link
Contributor

LGTM

@Davidson-Souza
Copy link
Member

Davidson-Souza commented Oct 18, 2025

What yall think of adding linting for justfile into CI ?

Is there linting for justfies?

@jaoleal jaoleal self-assigned this Oct 20, 2025
doc/justfile.md Outdated
# Just Recipes on Floresta.


In floresta we love to use [Just](https://just.systems/man/en/), its fast and a easy way to maintain scripts in the project.
Copy link
Member

Choose a reason for hiding this comment

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

The unresolved cold-hearted nit from Davidson

Copy link
Member

Choose a reason for hiding this comment

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

It's still unresolved

@jaoleal
Copy link
Collaborator Author

jaoleal commented Oct 29, 2025

done @Davidson-Souza @JoseSK999

@jaoleal jaoleal force-pushed the justfile_groups branch 2 times, most recently from e803a24 to 1584f15 Compare December 13, 2025 11:50
@jaoleal
Copy link
Collaborator Author

jaoleal commented Dec 13, 2025

Applied suggestions on 1584f15

The diff is really messed but im almost certain that i didnt overwrote anything

@jaoleal
Copy link
Collaborator Author

jaoleal commented Dec 16, 2025

Some review here guys ? @Davidson-Souza @JoseSK999 @moisesPompilio

@jaoleal
Copy link
Collaborator Author

jaoleal commented Dec 17, 2025

What... the docker build have random failure ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code quality Generally improves code readability and maintainability enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants