[BACKLOG-45776][BACKLOG-49103] translation for English to Spanish (es_AR)#10462
Draft
ddiroma wants to merge 1 commit intopentaho:masterfrom
Draft
[BACKLOG-45776][BACKLOG-49103] translation for English to Spanish (es_AR)#10462ddiroma wants to merge 1 commit intopentaho:masterfrom
ddiroma wants to merge 1 commit intopentaho:masterfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR is a "translation POC round 4" focused on maintaining the order of key/value pairs in Spanish (Argentina) locale (es_AR) .properties files. It adds new auto-generated translation files and reorganizes existing ones with updated headers and restructured key ordering.
Changes:
- Adds new
messages_es_AR.propertiesfiles for numerous packages that previously had none. - Reorganizes existing
messages_es_AR.propertiesfiles with new auto-generated headers and reordered keys. - Adds new translation keys that were missing from prior versions.
Reviewed changes
Copilot reviewed 187 out of 410 changed files in this pull request and generated 6 comments.
| File | Description |
|---|---|
messages_es_AR.properties (all files) |
New or updated Spanish (Argentina) translations with maintained key/value ordering |
messages_en_US.properties (cluster) |
Minor fix: escaped apostrophe corrected in English message |
Comments suppressed due to low confidence (2)
engine/src/main/resources/org/pentaho/di/trans/steps/setvalueconstant/messages/messages_es_AR.properties:1
- The value contains a replacement character (garbled character:
N<0xFFFD>{0}). It should likely beLínea N°{0}orLínea N°: {0}.
# Auto-generated Spanish (Argentina) file
engine/src/main/resources/org/pentaho/di/trans/steps/fieldschangesequence/messages/messages_es_AR.properties:1
- The value contains a replacement character (garbled encoding:
N<0xFFFD>{0}). This is the same corruption as seen inSetValueConstant.Log.NLigne. It should beLínea N°{0}.
# Auto-generated Spanish (Argentina) file
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
engine/src/main/resources/org/pentaho/di/trans/steps/syslog/messages/messages_es_AR.properties
Outdated
Show resolved
Hide resolved
...e/src/main/resources/org/pentaho/di/trans/steps/mergejoin/messages/messages_es_AR.properties
Outdated
Show resolved
Hide resolved
...c/main/resources/org/pentaho/di/trans/steps/dynamicsqlrow/messages/messages_es_AR.properties
Show resolved
Hide resolved
...ntaho/di/trans/steps/symmetriccrypto/symmetriccryptotrans/messages/messages_es_AR.properties
Show resolved
Hide resolved
...ntaho/di/trans/steps/symmetriccrypto/symmetriccryptotrans/messages/messages_es_AR.properties
Show resolved
Hide resolved
...ain/resources/org/pentaho/di/trans/steps/pgpdecryptstream/messages/messages_es_AR.properties
Show resolved
Hide resolved
This comment has been minimized.
This comment has been minimized.
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 188 out of 411 changed files in this pull request and generated 10 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
...pentaho/di/trans/steps/symmetriccrypto/symmetricalgorithm/messages/messages_es_AR.properties
Show resolved
Hide resolved
...ain/resources/org/pentaho/di/trans/steps/pgpencryptstream/messages/messages_es_AR.properties
Show resolved
Hide resolved
...ain/resources/org/pentaho/di/trans/steps/pgpdecryptstream/messages/messages_es_AR.properties
Show resolved
Hide resolved
core/src/main/resources/org/pentaho/di/core/util/messages/messages_es_AR.properties
Show resolved
Hide resolved
engine/src/main/resources/org/pentaho/di/job/entries/telnet/messages/messages_es_AR.properties
Show resolved
Hide resolved
...src/main/resources/org/pentaho/di/trans/steps/sortedmerge/messages/messages_es_AR.properties
Outdated
Show resolved
Hide resolved
...c/main/resources/org/pentaho/di/trans/steps/dynamicsqlrow/messages/messages_es_AR.properties
Show resolved
Hide resolved
...ain/resources/org/pentaho/di/trans/steps/setvalueconstant/messages/messages_es_AR.properties
Outdated
Show resolved
Hide resolved
.../resources/org/pentaho/di/job/entries/webserviceavailable/messages/messages_es_AR.properties
Show resolved
Hide resolved
...rc/main/resources/org/pentaho/di/trans/steps/processfiles/messages/messages_es_AR.properties
Outdated
Show resolved
Hide resolved
ddiroma
changed the title
This comment has been minimized.
This comment has been minimized.
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 189 out of 412 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
engine/src/main/resources/org/pentaho/di/imp/messages/messages_es_AR.properties
Outdated
Show resolved
Hide resolved
...pentaho/di/trans/steps/symmetriccrypto/symmetricalgorithm/messages/messages_es_AR.properties
Show resolved
Hide resolved
| @@ -11,6 +13,7 @@ ProcessFilesDialog.TargetFileNameField.Label=Campo de archivo destino | |||
| ProcessFiles.Log.SourceFileCopied=El archivo fuente [{0}] fue copiado a [{1}] | |||
| ProcessFilesDialog.Simulate.Tooltip=Si selecciona esta opci\u00F3n, los archivos no ser\u00E1n copiados/movidos/borrados.\nS\u00F3lo se verificar\u00E1n los archivos (existencia, directorio padre) y se mostrar\u00E1 un mensaje. | |||
| ProcessFiles.Error.SourceFileNotFile=El archivo fuente [{0}] no es un archivo (probablemente un directorio. | |||
There was a problem hiding this comment.
The message has an unmatched opening parenthesis, which will show up in the UI/logs. Close the parenthesis to keep the text well-formed.
Suggested change
| ProcessFiles.Error.SourceFileNotFile=El archivo fuente [{0}] no es un archivo (probablemente un directorio. | |
| ProcessFiles.Error.SourceFileNotFile=El archivo fuente [{0}] no es un archivo (probablemente un directorio). |
...resources/org/pentaho/di/trans/steps/fieldschangesequence/messages/messages_es_AR.properties
Outdated
Show resolved
Hide resolved
This comment has been minimized.
This comment has been minimized.
|
0 New Issues
0 Fixed Issues
0 Accepted Issues
No data about coverage (25.60% Estimated after merge)
Collaborator
|
📦 Vulnerable Dependencies✍️ Summary
🔬 Research DetailsDescription: This issue affects Apache Commons Lang: Starting with commons-lang:commons-lang 2.0 to 2.6, and, from org.apache.commons:commons-lang3 3.0 before 3.18.0. The methods ClassUtils.getClass(...) can throw StackOverflowError on very long inputs. Because an Error is usually not handled by applications and libraries, a Users are recommended to upgrade to version 3.18.0, which fixes the issue. Note:Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system. |
Collaborator
❌ Build failed in 1h 34m 32sBuild command: mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl \
core,engine,plugins,plugins/aggregate-rows/core,plugins/avro-format/core,plugins/connected-to-repository/impl,plugins/connections/ui,plugins/core/impl,plugins/core/ui,plugins/credit-card-validator/impl,plugins/drools/core,plugins/dummy/core,plugins/edi2xml/impl,plugins/email-messages/impl,plugins/engine-configuration/impl,plugins/excel/core,plugins/export-repository/impl,plugins/file-open-save-new/api,plugins/file-open-save-new/core,plugins/ftp-delete/impl,plugins/ftps/impl,plugins/get-file-sftp/impl,plugins/get-file-with-ftp/impl,plugins/get-previous-row-field/core,plugins/get-repository-names/impl,plugins/google-analytics/core,plugins/gpload/core,plugins/hl7/core,plugins/infobright-bulk-loader/impl,plugins/ivw-bulk-loader/impl,plugins/json/core,plugins/kafka/core,plugins/ldap/impl,plugins/mail-job/impl,plugins/mail-validator/impl,plugins/mail/impl,plugins/meta-inject/impl,plugins/mondrianinput/impl,plugins/monet-db-bulk-loader/impl,plugins/monet-db-bulk-loader/ui,plugins/ms-access/impl,plugins/ms-access/ui,plugins/msg-box-info/impl,plugins/mysql-bulk-loader/impl,plugins/mysql-bulk-loader/ui,plugins/oracle-bulk-loader/impl,plugins/oracle-bulk-loader/ui,plugins/pentaho-reporting/impl,plugins/postgresql-db-bulk-loader/impl,plugins/pur/core,plugins/put-a-file-with-ftp/impl,plugins/put-file-sftp/impl,plugins/random-cc-number-generator/impl,plugins/repositories/core,plugins/rest/core,plugins/rss/impl,plugins/s3-vfs/core,plugins/s3csvinput/core,plugins/salesforce/core,plugins/sap/core,plugins/sftpput/impl,plugins/shapefilereader/core,plugins/streaming/impls/jms,plugins/streaming/impls/mqtt,plugins/terafast-bulk-loader/impl,plugins/xml/core,plugins/yaml-input/impl,ui👌 All tests passed! Tests run: 6570, Failures: 0, Skipped: 25 Test Results ℹ️ This is an automatic message |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.