commercetest / nlnet

Analysis of the opensource codebases of NLnet sponsored projects.
MIT License
0 stars 0 forks source link

Including and checking the repo urls for non-github repos #61

Closed tnzmnjm closed 2 months ago

tnzmnjm commented 2 months ago

Ensure Cross-Platform Compatibility: Verify and adjust the current logic used for parsing owner/repository information from GitHub repositories to ensure compatibility with other repository hosting platforms.

tnzmnjm commented 2 months ago
tnzmnjm commented 2 months ago

I expand test coverage for get_base_repo_url with multi-platform scenarios:

tnzmnjm commented 2 months ago
tnzmnjm commented 2 months ago

Enhanced repository cloning and error logging functionality:

Introduced several improvements to the script responsible for cloning GitHub repositories and managing errors during the process.

Key Changes:

  1. Directory Structure by Domain: Modified the cloning path to create subdirectories based on the 'repodomain' column from the input DataFrame.

  2. Enhanced Error Handling:

    • Implemented capturing of detailed error messages during the repository cloning process.
    • Added functionality to write these error messages, along with the repository URL, to a dedicated error log file (data/error_log.txt).
  3. DataFrame Updates:

    • Introduced a 'clone_status' column to track the success or failure of each cloning attempt, enhancing the script's reporting capabilities.

Details:

This commit is related to the issues #61 and #56