Skip to content

Conversation

@edwardchalstrey1
Copy link
Member

@edwardchalstrey1 edwardchalstrey1 commented Nov 13, 2025

Copy link
Member

@tturocy tturocy left a comment

Choose a reason for hiding this comment

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

I have made some changes for clarification. Passing on to the rest of the team for their input based on their own experiences in getting set up for development.

@tturocy
Copy link
Member

tturocy commented Nov 13, 2025

I have requested reviews by all current devs for their input as to the completeness of the advice on installing and using pre-commit and writing commit messages.

Copy link
Member

@rahulsavani rahulsavani left a comment

Choose a reason for hiding this comment

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

This seems to cover the main relevant information I can think of.

Copy link
Collaborator

@StephenPasteris StephenPasteris left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@edwardchalstrey1 edwardchalstrey1 moved this from In review to In progress in Ed Chalstrey Gambit priorities Nov 28, 2025
Copy link
Contributor

@d-kad d-kad left a comment

Choose a reason for hiding this comment

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

Docs look good to me!

@edwardchalstrey1 edwardchalstrey1 moved this from In progress to In review in Ed Chalstrey Gambit priorities Nov 28, 2025
@edwardchalstrey1 edwardchalstrey1 merged commit e389a88 into master Nov 28, 2025
1 check passed
@github-project-automation github-project-automation bot moved this from In review to Done in Ed Chalstrey Gambit priorities Nov 28, 2025
@edwardchalstrey1 edwardchalstrey1 deleted the add-pre-commit branch November 28, 2025 13:50
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.

6 participants