Skip to content

Update Contribution.md#7238

Merged
saurabhraghuvanshii merged 3 commits intolayer5io:masterfrom
CodexRaunak:update-contributing-docs
Dec 8, 2025
Merged

Update Contribution.md#7238
saurabhraghuvanshii merged 3 commits intolayer5io:masterfrom
CodexRaunak:update-contributing-docs

Conversation

@CodexRaunak
Copy link
Copy Markdown
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.

…ribution docs

Signed-off-by: Raunak Madan <madanraunak24@gmail.com>
Copy link
Copy Markdown
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
Copy Markdown
Member

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 <madanraunak24@gmail.com>
@CodexRaunak CodexRaunak force-pushed the update-contributing-docs branch from c3a93bc to b6a7748 Compare December 6, 2025 23:43
@l5io
Copy link
Copy Markdown
Member

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
Copy Markdown
Member

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