Skip to content

Conversation

@tdhock
Copy link

@tdhock tdhock commented Feb 10, 2026

Hi @cderv
As you suggested in #14012
here is a new file for fr-CA.
Does that look ok to you?
Merci !

@posit-snyk-bot
Copy link
Collaborator

posit-snyk-bot commented Feb 10, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@cderv
Copy link
Collaborator

cderv commented Feb 10, 2026

Thanks for the new PR. Yes this is good !

Though I was not precise enough: there should be inheritance and merge in the sense that

  • _language-fr-CA.yml will be read
  • _language-fr.yml will be read

Both will be merged, meaning the former can only include what needs to be override in the later.

That would simplify maintenance.

To be tested for confirmation.

@tdhock
Copy link
Author

tdhock commented Feb 10, 2026

Thanks for your quick feedback.
I removed everything except the two author lines.
I don’t know how to write a test though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants