8000 🌐 Tamil translations - initial setup by arunppsg · Pull Request #5564 · fastapi/fastapi · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
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.

Sign up for GitHub

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

Jump to bottom

🌐 Tamil translations - initial setup #5564

Merged
merged 6 commits into from
Mar 6, 2023
Merged

🌐 Tamil translations - initial setup #5564

merged 6 commits into from
Mar 6, 2023

Conversation

arunppsg
Copy link
Contributor
@arunppsg arunppsg commented Oct 31, 2022 •
edited
Loading

#5434

In this PR, I have added setup files for tamil translation of docs.

@codecov
Copy link
codecov bot commented Oct 31, 2022 •
edited
Loading

Codecov Report

Patch and project coverage have no change.

Comparison is base (cf73051) 100.00% compared to head (ec42f6c) 100.00%.

❗ Current head ec42f6c differs from pull request most recent head 23335ca. Consider uploading reports for the commit 23335ca to get more accurate results

Additional details and impacted files 
@@            Coverage Diff            @@
##            master     #5564   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          540       540           
  Lines        13969     13946   -23     
=========================================
- Hits         13969     13946   -23
Impacted Files Coverage Δ
fastapi/routing.py 100.00% <0.00%> (ø)
tests/test_datastructures.py 100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@arunppsg arunppsg mentioned this pull request 8000 Nov 1, 2022
Closed
@tiangolo
Copy link
Member
tiangolo commented Nov 7, 2022

It seems that you removed or didn't commit some of the needed generated files and that's why CI isn't passing.

@github-actions
Copy link
Contributor
github-actions bot commented Nov 8, 2022

📝 Docs preview for commit bf3c203 at: https://636a62c307db1d199b4cc10a--fastapi.netlify.app

@arunppsg
Copy link
Contributor Author
arunppsg commented Nov 8, 2022

It seems that you removed or didn't commit some of the needed generated files and that's why CI isn't passing.

ta was not supported as a language in jinja template and I didn't notice it earlier. I updated it by changing language code to en as mentioned in docs here. Now the builds are passing.

@github-actions
Copy link
Contributor
github-actions bot commented Nov 8, 2022

📝 Docs preview for commit ecd67fc at: https://636a6ae0ed372f225492d3b0--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commit 9b8e15d at: https://639ce384ecf1fd06559b406b--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commit 6f2515a at: https://63bd5cc12d54bb24e7f89f71--fastapi.netlify.app

@github-actions
Copy link
Contributor
github-actions bot commented Mar 4, 2023

📝 Docs preview for commit a4102bb at: https://64033efeac95be336b619b1a--fastapi.netlify.app

@tiangolo tiangolo added the lang-ta Tamil translations label Mar 6, 2023
@tiangolo tiangolo changed the title Tamil docs - initial commit setting up 🌐 Tamil translations - initial setup Mar 6, 2023
@tiangolo
Copy link
Member
tiangolo commented Mar 6, 2023

Great, thanks @gusty1g! 🚀

@github-actions
Copy link
Contributor
github-actions bot commented Mar 6, 2023

📝 Docs preview for commit 23335ca at: https://640615c420abd00084241f3e--fastapi.netlify.app

@tiangolo tiangolo merged commit 639cf34 into fastapi:master Mar 6, 2023
mahenzon pushed a commit to mahenzon/fastapi that referenced this pull request Mar 11, 2023
@arunppsg @tiangolo @mahenzon
🌐 Tamil translations - initial setup (fastapi#5564)
830c8e1 
Co-authored-by: Sebastián Ramírez <tiangolo@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
investigate lang-all Translations lang-ta Tamil translations
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@arunppsg @tiangolo
0