Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat:add readme for mods #398

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

atybdot
Copy link

@atybdot atybdot commented Dec 31, 2024

readme component that fetch the readme of respective mod and display it.

Features it add :

  1. render readme file of each mod
  2. link to the respective readme file

screenshots
image
image

@NOCanoa NOCanoa requested a review from mauro-balades January 1, 2025 16:25
@atybdot
Copy link
Author

atybdot commented Jan 2, 2025

@mr-cheff i didn't knew there was a pnpm-lock file.
I thought i created it. and vs-code was recommending me that i should only have one lock per project, otherwise there may be conflicts. that's why i deleted it.

@mauro-balades
Copy link
Member

Im still trying to figure out if we really want READMEs...

@atybdot
Copy link
Author

atybdot commented Jan 2, 2025

i was going through code-base,i found a to-do:add readme.
That's why i did it.Although it's up to you to decide, i recommend adding this, could be useful for detailed description of mods ,dos and dont's etc..

@NOCanoa NOCanoa added the mr. m ⌚ waithing for cheff opinion/confirmation label Jan 3, 2025
@NOCanoa NOCanoa added the invalid This doesn't seem right label Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right mr. m ⌚ waithing for cheff opinion/confirmation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants