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

Add Galician translation #8226

Merged
merged 5 commits into from Jun 27, 2014

Conversation

Projects
None yet
3 participants
@barcia
Contributor

barcia commented Jun 23, 2014

Add Galician translation for 3a762c3

Iván Barcia
Add Galician translation
Add Galician translation for 3a762c3

@redmunds redmunds self-assigned this Jun 23, 2014

@redmunds

This comment has been minimized.

Show comment
Hide comment
@redmunds

redmunds Jun 23, 2014

Contributor

@ivarcia Thanks for ageeing to the CLA and contributing to Brackets. Take a look at the instructions in the nls/README.md file for how to add a translation to Brackets -- there are a few more files that you need to update (including the README.md file itself).

It looks like you are contributing via the github.com web site, so be sure to read this section in particular.

Contributor

redmunds commented Jun 23, 2014

@ivarcia Thanks for ageeing to the CLA and contributing to Brackets. Take a look at the instructions in the nls/README.md file for how to add a translation to Brackets -- there are a few more files that you need to update (including the README.md file itself).

It looks like you are contributing via the github.com web site, so be sure to read this section in particular.

@redmunds

View changes

Show outdated Hide outdated src/nls/gl/strings.js
@barcia

This comment has been minimized.

Show comment
Hide comment
@barcia

barcia Jun 25, 2014

Contributor

I think that now all is ok.

Contributor

barcia commented Jun 25, 2014

I think that now all is ok.

@ingorichter

This comment has been minimized.

Show comment
Hide comment
@ingorichter

ingorichter Jun 25, 2014

Contributor

@ivarcia Thanks for your contribution. In order to include Galician in the 0.41 release, could you please update the copyright year to 2014 in src/nls/gl/strings.js and create a PR based on the release branch, since this is the branch we use to create the final build. That would be much appreciated. Thanks.

Contributor

ingorichter commented Jun 25, 2014

@ivarcia Thanks for your contribution. In order to include Galician in the 0.41 release, could you please update the copyright year to 2014 in src/nls/gl/strings.js and create a PR based on the release branch, since this is the branch we use to create the final build. That would be much appreciated. Thanks.

@redmunds

This comment has been minimized.

Show comment
Hide comment
@redmunds

redmunds Jun 25, 2014

Contributor

@ivarcia It's too late to get this into Release 0.41, so you can keep this targeted for the master branch. As soon as you update the copyright years, we'll merge this into master.

Contributor

redmunds commented Jun 25, 2014

@ivarcia It's too late to get this into Release 0.41, so you can keep this targeted for the master branch. As soon as you update the copyright years, we'll merge this into master.

@barcia

This comment has been minimized.

Show comment
Hide comment
@barcia

barcia Jun 26, 2014

Contributor

Ok, the copyright year was updated.
Sorry for the inconveniences and thanks for your help @redmunds and @ingorichter , I'm a little newbie at this, but I am happy to translate this great application to my language.

Contributor

barcia commented Jun 26, 2014

Ok, the copyright year was updated.
Sorry for the inconveniences and thanks for your help @redmunds and @ingorichter , I'm a little newbie at this, but I am happy to translate this great application to my language.

@redmunds

This comment has been minimized.

Show comment
Hide comment
@redmunds

redmunds Jun 27, 2014

Contributor

Looks good. Merging.

Contributor

redmunds commented Jun 27, 2014

Looks good. Merging.

redmunds added a commit that referenced this pull request Jun 27, 2014

@redmunds redmunds merged commit 91a6b4d into adobe:master Jun 27, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment