Skip to content

Unify all code comments to English#14

Merged
vincentg merged 4 commits intomasterfrom
copilot/fix-13
Jul 24, 2025
Merged

Unify all code comments to English#14
vincentg merged 4 commits intomasterfrom
copilot/fix-13

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jul 24, 2025

This PR addresses the issue of mixed language comments in the codebase by translating all French comments to English, ensuring consistency across all source files.

Changes Made

The following French comments were identified and translated to English in src/check_snmp_load.c:

  1. Line 356: /* Allocation de 10 en 10 *//* Allocate in chunks of 10 */
  2. Line 379: /* Test si ce n'est pas une exception *//* Test if it's not an exception */
  3. Line 384: /* Et l'on avance dans la MIB :) *//* And we advance in the MIB :) */
  4. Line 392: /* une exception , donc stop *//* an exception, so stop */
  5. Lines 417-422: Function documentation block translated from French to English:
    • utilise la structure process, cherche l'occupation memoire et afficheuses the process structure, searches for memory occupation and displays
    • taille de la structure (nb d'elements)size of the structure (number of elements)

Verification

  • Build verification: All executables compile successfully without errors
  • Comprehensive search: Confirmed no remaining French comments in any source files
  • Functionality preserved: No functional code changes, only comment translations
  • Translation accuracy: All translations maintain original meaning and technical context

The changes are minimal and surgical, affecting only comment text while preserving all code functionality. The repository now has unified English comments throughout all source files, improving maintainability and accessibility for international contributors.

Fixes #13.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

  • esm.ubuntu.com
    • Triggering command: /usr/lib/apt/methods/https (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copilot AI and others added 2 commits July 24, 2025 05:09
Co-authored-by: vincentg <422251+vincentg@users.noreply.github.com>
Co-authored-by: vincentg <422251+vincentg@users.noreply.github.com>
Copilot AI changed the title [WIP] Unify all code comments to English Unify all code comments to English Jul 24, 2025
Copilot AI requested a review from vincentg July 24, 2025 05:15
@vincentg vincentg marked this pull request as ready for review July 24, 2025 05:16
CMakeCache.txt Outdated
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This PR includes many files that should be ignored as this is from a commit before the .gitignore was created. Please remove from the branch all unnecessary files

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Removed all build artifacts from the repository. The branch now only contains the essential source files plus the .gitignore file and comment translations. Build artifacts are now properly ignored by the .gitignore rules. Fixed in commit fdb8aaa.

Co-authored-by: vincentg <422251+vincentg@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Jul 24, 2025

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

  • esm.ubuntu.com
    • Triggering command: /usr/lib/apt/methods/https (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI requested a review from vincentg July 24, 2025 05:24
@vincentg vincentg merged commit 2329cf1 into master Jul 24, 2025
1 check passed
@vincentg vincentg deleted the copilot/fix-13 branch July 24, 2025 05:27
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.

Unify all code comments to English

2 participants