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

Occitan updated #2267

Merged
merged 1 commit into from
Sep 5, 2016
Merged

Occitan updated #2267

merged 1 commit into from
Sep 5, 2016

Conversation

Quent-in
Copy link
Contributor

@Quent-in Quent-in commented Sep 5, 2016

Q A
Bug fix? yes
New feature? yes
BC breaks? yes
Deprecations? yes
Tests pass? yes
Documentation yes
Translation yes
Fixed tickets comma-separated list of tickets fixed by the PR, if any
License MIT

Line 347, I'm not sure about what I wrote to avoid and end of string started with "
I had to use the simple ' in the sentence.
Let me know if it was okay like so.

Line 347, I'm not sure about what I wrote to avoid and end of string started with "
I had to use the simple ' in the sentence.
Let me know if it was okay like so.
@nicosomb
Copy link
Member

nicosomb commented Sep 5, 2016

👍

@j0k3r
Copy link
Member

j0k3r commented Sep 5, 2016

Looks great ! Thanks :)

@j0k3r j0k3r merged commit e5b0909 into 2.1 Sep 5, 2016
@j0k3r j0k3r deleted the Quent-in-patch-1 branch September 5, 2016 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants