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

Remove the /lang directory #5158

Closed
wants to merge 3 commits into from
Closed

Remove the /lang directory #5158

wants to merge 3 commits into from

Conversation

teolemon
Copy link
Member

@teolemon teolemon commented Apr 17, 2021

What

#   Failed test 'footer_legal_link link - lang: pt - textid: informacao-legal -- file /lang/pt/texts/informacao-legal.html does not exist'
#   at t/lang.t line 52.

#   Failed test 'footer_legal_link link - lang: fr - textid: mentions-legales -- file /lang/fr/texts/mentions-legales.html does not exist'
#   at t/lang.t line 52.

#   Failed test 'footer_terms_link link - lang: fr - textid: conditions-d-utilisation -- file /lang/fr/texts/conditions-d-utilisation.html does not exist'
#   at t/lang.t line 52.

#   Failed test 'footer_terms_link link - lang: pt - textid: termos-de-utilizacao -- file /lang/pt/texts/termos-de-utilizacao.html does not exist'
#   at t/lang.t line 52.
# Looks like you failed 4 tests of 376.
t/lang.t ......................... 

Part of

  • Static content

@hangy
Copy link
Member

hangy commented Apr 17, 2021

How do you want to integrate this into develpoment environments, ie. Docker? Also, the GitHub workflows will need to be fixed. For example, the PR validation does try to build language (I assume to validate the translations):

perl -CS -I$BUILD_DIR/lib $BUILD_DIR/scripts/build_lang.pl

@stephanegigandet
Copy link
Contributor

@hangy : if we leave an empty lang directory, then build_lang.pl will work (the /po directory is still there).

@hangy
Copy link
Member

hangy commented Apr 19, 2021

@hangy : if we leave an empty lang directory, then build_lang.pl will work (the /po directory is still there).

Good idea, but don't we need some kind of submodule, so that developers can test locally easily?

@stephanegigandet
Copy link
Contributor

Good idea, but don't we need some kind of submodule, so that developers can test locally easily?

Maybe just a script that would do a git clone of openfoodfacts-web and link the /lang directory to the clone? Or maybe you have a better idea on how we could do it?

@sonarcloud
Copy link

sonarcloud bot commented Apr 22, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@stephanegigandet
Copy link
Contributor

stephanegigandet commented Apr 23, 2021

So we'll need to move those tests to openfoodfacts-web:

#   Failed test 'footer_legal_link link - lang: pt - textid: informacao-legal -- file /lang/pt/texts/informacao-legal.html does not exist'
#   at t/lang.t line 52.

#   Failed test 'footer_legal_link link - lang: fr - textid: mentions-legales -- file /lang/fr/texts/mentions-legales.html does not exist'
#   at t/lang.t line 52.

#   Failed test 'footer_terms_link link - lang: fr - textid: conditions-d-utilisation -- file /lang/fr/texts/conditions-d-utilisation.html does not exist'
#   at t/lang.t line 52.

#   Failed test 'footer_terms_link link - lang: pt - textid: termos-de-utilizacao -- file /lang/pt/texts/termos-de-utilizacao.html does not exist'
#   at t/lang.t line 52.
# Looks like you failed 4 tests of 376.
t/lang.t ......................... 

@teolemon teolemon added the static content This lives at https://github.com/openfoodfacts/openfoodfacts-web label Apr 26, 2021
@teolemon teolemon changed the title Remove Lang Remove the /lang directory Jun 27, 2021
@stephanegigandet
Copy link
Contributor

Done in PR #6113

@stephanegigandet stephanegigandet deleted the remove-lang branch January 4, 2022 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
static content This lives at https://github.com/openfoodfacts/openfoodfacts-web
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants