Skip to content
This repository has been archived by the owner on Apr 25, 2024. It is now read-only.

README in Portuguese #184

Closed
wants to merge 8 commits into from
Closed

README in Portuguese #184

wants to merge 8 commits into from

Conversation

pedrocatalao
Copy link

No description provided.

@msftclas
Copy link

msftclas commented Oct 8, 2018

CLA assistant check
All CLA requirements met.

@RubenMateus
Copy link

Already added here, wanna change here? #163

@pedrocatalao
Copy link
Author

Already added here, wanna change here? #163

@RubenMateus actually, I believe this is the only pt-PT translation that was carefully submitted with no typos, using all the original formatting and using naturally formed sentences.

If it was me, I'd select this PR given the aforementioned reasons but I leave it for @bitcrazed to decide. 😊

@RubenMateus
Copy link

RubenMateus commented Oct 11, 2018

Well that's not what contributing to open-source is about, it not about my pull rquest is better than yours, if i have any typos in name, please tell me and i will fix it...
that's the goal of everyone contributing.

@pedrocatalao
Copy link
Author

I'm not saying mine is better, only that I was careful with the quality of this PR.
I actually missed your PR before creating mine but I believe you also missed #79 otherwise you'd have contributed to it instead of creating a new one based on the existing pt-BR README which is full of mistakes and flaws such as orinalmente and tem adotado also present on yours.

@RubenMateus
Copy link

agreed, i didnt see #79, so i will close this, thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants