-
Notifications
You must be signed in to change notification settings - Fork 6
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
chore(docs): added other essential open source documentation files #245
chore(docs): added other essential open source documentation files #245
Conversation
- Added `CONTRIBUTING.md` with development and PR guidelines - Added `CODE_OF_CONDUCT.md` based on Contributor Covenant - Added `SECURITY.md` with vulnerability reporting process - Added issue and PR templates - Updated `README.md` with documentation references and fixed lint errors - Updated `.gitignore` file to ignore VSCode workspace settings
.github/ISSUE_TEMPLATE/bug_report.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @BSalaeddin, done
README.md
Outdated
![upup_logo_dark](https://github.com/DevinoSolutions/upup/assets/43147238/b5477db9-cb23-43c7-8c12-518beb31af53) | ||
|
||
<br/> | ||
An open-source, free-to-use NPM component that easily handles your file upload needs with seamless DigitalOcean Spaces, Amazon S3, Google Drive and OneDrive integration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggestion:
Given that the package relies on JSX and React hooks, it would be more precise to describe it as a "React component library" rather than a generic "NPM component," ensuring the description aligns with its core technology stack.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @BSalaeddin , done
- updated README to describe package as a "React component library" rather than a generic "NPM component," ensuring the description aligns with its core technology stack - included field for commit hash in bug report
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
CONTRIBUTING.md
with development and PR guidelinesCODE_OF_CONDUCT.md
based on Contributor CovenantSECURITY.md
with vulnerability reporting processREADME.md
with documentation references and fixed lint errors.gitignore
file to ignore VSCode workspace settings