search

google / flutter-mediapipe

Apache License 2.0
212 stars 14 forks source link

Contributing guide #59

Open craiglabenz opened 5 months ago

craiglabenz commented 5 months ago

Adds behind-the-scenes info needed to operate the repository and get things working during local development.

Seemingly as the first contributor, would you mind test-driving these instructions, @charlieforward9?

Resolves #17.

charlieforward9 commented 5 months ago

Lets do it ❗️

charlieforward9 commented 5 months ago

@craiglabenz

Checking out the google/mediapipe repository

The organization is now https://github.com/google-ai-edge/mediapipe , not google. This puts this repo in a weird spot... haha

charlieforward9 commented 5 months ago

@craiglabenz

About halfway through the guide, expect to have it done soon. I fixed a few typos.

I currently use the Prettier formatter. Is there a specific formatting method I should use for this repository?

craiglabenz commented 5 months ago

The organization is now https://github.com/google-ai-edge/mediapipe , not google. This puts this repo in a weird spot... haha

I knew that change was coming, but hadn't expected it this soon! I'll make updates.

charlieforward9 commented 5 months ago

I currently use the Prettier formatter. Is there a specific formatting method I should use for this repository?

@craiglabenz

EDIT: I found the Style Guide

craiglabenz commented 5 months ago

For Dart formatting in this library, I let the Dart extension in VS Code auto-format everything, and then I make sure the analysis server doesn't have any warnings. The Flutter repo's own style guide is unique and we don't need to follow it here.

iarshguleria commented 5 months ago

Hi @craiglabenz @charlieforward9 ,since I am also closely following this repo, I could also offer some help incase you need with formatting or other stuff