Skip to content

Conversation

@CodexRaunak
Copy link
Contributor

Description

  • Documented info about all the make targets in the make file
  • Documented use of all environment variables

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Copy link
Member

@leecalcote leecalcote left a comment

Choose a reason for hiding this comment

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

Please consider the proposed changes in this PR - #7229

Instead of listing all of the make targets here, please offer instructions to simply execute make to see the full list with descriptions of their purpose, instead.

@l5io
Copy link
Contributor

l5io commented Dec 6, 2025

🚀 Preview for commit 02feabf at: https://6934b24490262bc5316555e5--layer5.netlify.app

…pdate description of env variables

Signed-off-by: Raunak Madan <[email protected]>
@CodexRaunak CodexRaunak force-pushed the update-contributing-docs branch from c3a93bc to b6a7748 Compare December 6, 2025 23:43
@l5io
Copy link
Contributor

l5io commented Dec 6, 2025

🚀 Preview for commit b6a7748 at: https://6934c2c7e62416cc5e8cc7cb--layer5.netlify.app

@saurabhraghuvanshii saurabhraghuvanshii merged commit 376eedc into layer5io:master Dec 8, 2025
4 checks passed
@saurabhraghuvanshii
Copy link
Contributor

Thanks @CodexRaunak lgtm

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