Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.