Skip to content

Correction for French sentence "open"#225

Merged
Stypox merged 3 commits intoStypox:masterfrom
AgentSteel:master
Jul 26, 2024
Merged

Correction for French sentence "open"#225
Stypox merged 3 commits intoStypox:masterfrom
AgentSteel:master

Conversation

@AgentSteel
Copy link
Contributor

Correction for French sentence "open"

correction for french verb "ouvrir"
@AgentSteel AgentSteel marked this pull request as draft July 25, 2024 08:49
@AgentSteel AgentSteel marked this pull request as ready for review July 25, 2024 08:49
@AgentSteel AgentSteel marked this pull request as draft July 25, 2024 10:33
@AgentSteel AgentSteel marked this pull request as ready for review July 25, 2024 10:34
@@ -1,2 +1,2 @@
query:
- ouvre<r|z?>|lance<r|z?>|charge<r|z?>|execute<r|z?> (l? app|application)? .what.
- ouvrir<?>|ouvre<z?>|lance<r|z?>|charge<r|z?>|execute<r|z?> (l? app|application)? .what.
Copy link
Owner

Choose a reason for hiding this comment

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

<?> is not valid, but I guess you wanted to make the r optional? So the options would be ouvri, ouvrir, ouvre, ouvrez, is that correct? Also, what about the other words (lance<r|z?>, charge<r|z?>, execute<r|z?>), are these three options valid or is something missing: e.g. lance, lancer, lancez?
Note that if there is an additional incorrect word it's not an issue, but it's important that the correct words are all there.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hello, sorry for the mistake

the word "ouvrer" in french does not exist, it is "ouvrir".

to open = ouvrir

(but "ouvre" and "ouvrez" are valid)

All the others are OK too.

So the correction would be : ouvrir|ouvre<z?>|lance<r|z?> ....

Copy link
Owner

Choose a reason for hiding this comment

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

Ok, thank you!

correction
@Stypox Stypox merged commit 73d22af into Stypox:master Jul 26, 2024
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.

2 participants