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

PR for docs [renewed] #220

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from
Open

PR for docs [renewed] #220

wants to merge 4 commits into from

Conversation

joeyouss
Copy link

No description provided.

@@ -8,6 +8,7 @@ Google Summer Of Code Project under CCExtractor Development
[![GPLv3 license](https://img.shields.io/badge/License-GPLv3-blue.svg)](https://github.com/ccextractor/Rekognition/blob/master/LICENSE)

---
![bannerimage](https://github.com/CCExtractor/Rekognition/banner.png)
Copy link
Collaborator

Choose a reason for hiding this comment

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

image hasn’t been pushed

Copy link
Collaborator

Choose a reason for hiding this comment

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

create a folder named assets and move the image there
you will need to provide an exception to ignoring .png files in gitignore


As beginners, navigating the codebase and finding your way out of the documentation can become difficult. This page will help you understand everything about contributing to howdoi and the best practices in open source as well.

## Contributing to Rekognition
Copy link
Collaborator

Choose a reason for hiding this comment

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

this should start up with setting up dev environment locally
add link to slack in case someone needs help with setup process

Point them to existing APIs and ask them to try these APIs for themselves. If they find any bugs in the process and/or can come up with a feature request ask them to create an issue for the same

Then point to them the already open issues and provide guidance on how to submit a PR

@@ -1,3 +1,33 @@
Table of contents:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add a link to contributing.md in readme

@@ -8,6 +8,7 @@ Google Summer Of Code Project under CCExtractor Development
[![GPLv3 license](https://img.shields.io/badge/License-GPLv3-blue.svg)](https://github.com/ccextractor/Rekognition/blob/master/LICENSE)
Copy link
Collaborator

Choose a reason for hiding this comment

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

add usage command for each API

@PulkitMishra
Copy link
Collaborator

@joeyouss make changes to incorporate new api added in #222

@PulkitMishra
Copy link
Collaborator

what's the status on this one @joeyouss ?
Any blockers?

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.

2 participants