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

Typo and grammatical correction in French ASVS V4 #1942

Merged
merged 19 commits into from
May 2, 2024
Merged

Typo and grammatical correction in French ASVS V4 #1942

merged 19 commits into from
May 2, 2024

Conversation

Aif4thah
Copy link
Contributor

Hello,

some additional corrections concerning the French version.

especially for "Bytecode" -> "Code Intermédiaire"

Best regards

@Aif4thah
Copy link
Contributor Author

The URL checker has failed, but I have not edit any link

@tghosth
Copy link
Collaborator

tghosth commented May 2, 2024

Hi @Aif4thah, thanks for your work on this.

Will there be any more fixes which you want to include? I'd rather get them all done together so that I can ask someone to review them all in one go. Thanks!

@Aif4thah
Copy link
Contributor Author

Aif4thah commented May 2, 2024

Hi @Aif4thah, thanks for your work on this.

Will there be any more fixes which you want to include? I'd rather get them all done together so that I can ask someone to review them all in one go. Thanks!

I understand, I will try to review all the files and make a final PR

@Aif4thah
Copy link
Contributor Author

Aif4thah commented May 2, 2024

It’s done, the PR can now be reviewed. thx

@tghosth
Copy link
Collaborator

tghosth commented May 2, 2024

Thanks @Aif4thah

@clallier94 @inaz0 @SPoint42 @Marx314 Please review and let me know if there are any questions

@SPoint42
Copy link
Contributor

SPoint42 commented May 2, 2024

ok for me

@inaz0
Copy link
Contributor

inaz0 commented May 2, 2024

Ok for me

@tghosth tghosth merged commit b7e3870 into OWASP:master May 2, 2024
1 of 2 checks passed
@tghosth
Copy link
Collaborator

tghosth commented May 2, 2024

Thanks folks!

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.

4 participants