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

docs: ✏️ Adds summary to resources in readme #2

Merged
merged 2 commits into from Feb 5, 2020

Conversation

@marciofmjr
Copy link
Contributor

marciofmjr commented Jan 27, 2020

No description provided.

@marciofmjr marciofmjr requested a review from lfreneda Jan 27, 2020
Copy link
Member

lfreneda left a comment

Se o sumário for conter apenas os recursos, deve ficar na seção de "Usage".

Se for pra ficar no topo, deve conter todas as seções: Installation, Playground, Usage, Pull Request and License.

Faz sentido?

@marciofmjr

This comment has been minimized.

Copy link
Contributor Author

marciofmjr commented Jan 28, 2020

Se o sumário for conter apenas os recursos, deve ficar na seção de "Usage".

Se for pra ficar no topo, deve conter todas as seções: Installation, Playground, Usage, Pull Request and License.

Faz sentido?

eu pensei em colocar todas as seções, mas como elas estão muito pequenas, da pra ver as seções na mesma página sem rolar, por isso, achei que não seria necessário no momento.. o que você acha ?

@lfreneda

This comment has been minimized.

Copy link
Member

lfreneda commented Feb 5, 2020

Vamos ver como fica e qualquer coisa depois a gente altera :)

@lfreneda lfreneda self-requested a review Feb 5, 2020
Copy link
Member

lfreneda left a comment

👌

@lfreneda lfreneda merged commit baecf56 into FieldControl:master Feb 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.