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

Update French Principles and Glossary files #84

Merged
merged 10 commits into from
Aug 14, 2024
Merged

Update French Principles and Glossary files #84

merged 10 commits into from
Aug 14, 2024

Conversation

juliafmorgado
Copy link
Contributor

@juliafmorgado juliafmorgado commented Aug 13, 2024

Signed-off-by: Julia Furst Morgado [email protected]

Related to #77

@scottrigby
Copy link
Member

thanks so much! would you please fix the DCO sign-off? The DCO bot check gives step by step instructions. Ping me if i can help with that.

@scottrigby scottrigby mentioned this pull request Aug 14, 2024
Signed-off-by: Julia Furst Morgado <[email protected]>
Signed-off-by: Julia Furst Morgado <[email protected]>
@scottrigby
Copy link
Member

Thanks for fixing DCO - and for noticing this repo's CONTRIBUTING guidelines doens't mention that as a requirement! Our GitHub org's default contributing file does, but the repo-specific one overrides that, so we need to fix that. I opened an issue to track that:

Anyway, we now just need another French speaker to review this PR 😄

Copy link

@lrochette lrochette left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See my comments

i18n/GLOSSARY_fr.md Show resolved Hide resolved
i18n/GLOSSARY_fr.md Show resolved Hide resolved
i18n/GLOSSARY_fr.md Outdated Show resolved Hide resolved
i18n/GLOSSARY_fr.md Outdated Show resolved Hide resolved
i18n/GLOSSARY_fr.md Outdated Show resolved Hide resolved
i18n/PRINCIPLES_fr.md Outdated Show resolved Hide resolved
Implement lrochette's recommendations

Signed-off-by: Julia Furst Morgado <[email protected]>
Signed-off-by: Julia Furst Morgado <[email protected]>
Signed-off-by: Julia Furst Morgado <[email protected]>
Signed-off-by: Julia Furst Morgado <[email protected]>
Signed-off-by: Julia Furst Morgado <[email protected]>
Signed-off-by: Julia Furst Morgado <[email protected]>
Signed-off-by: Julia Furst Morgado <[email protected]>
Signed-off-by: Julia Furst Morgado <[email protected]>
Copy link

@lrochette lrochette left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@scottrigby scottrigby left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work @juliafmorgado and @lrochette 🤝 🙌

@scottrigby scottrigby merged commit 8be046e into open-gitops:main Aug 14, 2024
1 check passed
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