Repository navigation
fix(naming): corrige grafia OminiRTkSync para OmniRTkSync - #25
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 56a4fcc9e4
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| (re.compile(r"9RTKSync|OmniRTKSync|OmniRTkSync|LiteLlmRTKSync"), "nome de produto literal"), | ||
| (re.compile(r"9Router|OmniRoute|LiteLLM"), "nome de gateway literal"), | ||
| (re.compile(r"9rtk-|ominirtk-|litellmrtk-"), "prefixo de container literal"), | ||
| (re.compile(r"9rtk-|omnirtk-|litellmrtk-"), "prefixo de container literal"), |
There was a problem hiding this comment.
Replacing the old alternatives removes the regression protection for the exact typo this change fixes: if a shared package module later contains OminiRTkSync, OminiRTKSync, or an ominirtk- container prefix, test_nenhum_modulo_escreve_identidade_a_mao now passes. Keep the obsolete forms as additional forbidden aliases so the typo cannot silently return outside identidade.py.
Useful? React with 👍 / 👎.
|
|
||
| # Nome do diretório de cada irmão ao lado deste repositório. | ||
| IRMAOS = ("9RTKSync", "OminiRTkSync", "LiteLlmRTKSync") | ||
| IRMAOS = ("9RTKSync", "OmniRTkSync", "LiteLlmRTKSync") |
There was a problem hiding this comment.
When an existing multi-repository workspace pulls this rename, Git updates the checkout contents but does not rename its OminiRTkSync directory. This new constant therefore makes test_irmaos_identicos treat that sibling as absent and skip every comparison; the analogous loop in test_rotas_simetricas.py silently continues as well. Discover the sibling under both directory names, or explicitly migrate/fail on the former name, so existing workspaces do not lose the cross-project parity checks.
Useful? React with 👍 / 👎.
1 participant
Corrige a grafia do nome: OminiRTkSync -> OmniRTkSync (havia um 'i' a mais depois do 'm').
omini_rtksync->omni_rtksync; comandos, imagem Docker (ghcr.io/pathbit/omnirtksync), cookies, volumes, redes, workflows, docs e testes.OminiRoute->OmniRoute.Validacao local
tools/valida_docs.py: 0 divergencias