Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Add Galician translation#8226

Merged
redmunds merged 5 commits intoadobe:masterfrom
barcia:patch-1
Jun 27, 2014
Merged

Add Galician translation#8226
redmunds merged 5 commits intoadobe:masterfrom
barcia:patch-1

Conversation

@barcia
Copy link
Copy Markdown
Contributor

@barcia barcia commented Jun 23, 2014

Add Galician translation for 3a762c3

Add Galician translation for 3a762c3
@redmunds redmunds self-assigned this Jun 23, 2014
@redmunds
Copy link
Copy Markdown
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.

Comment thread src/nls/gl/strings.js Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

New files should have a copyright date of the current year, so 2012 should be 2014.

@barcia
Copy link
Copy Markdown
Contributor Author

barcia commented Jun 25, 2014

I think that now all is ok.

@ingorichter
Copy link
Copy Markdown
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.

@redmunds
Copy link
Copy Markdown
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.

@barcia
Copy link
Copy Markdown
Contributor Author

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
Copy link
Copy Markdown
Contributor

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
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants