Skip to content

Conversation

@adarshaks91
Copy link
Contributor

@adarshaks91 adarshaks91 commented Sep 12, 2025

Description

While testing this PR in crowdcontrol, was getting the below error for AI testing methodology updates
Screenshot 2025-09-11 at 5 39 01 PM

Henceforth,

  • Point 0.1 submodule directory to sha commit in the past where the AI testing wasn’t changed (f8aa5884da64ffc3d000126dff067cd7abde3b6a)
  • Create a new submodule directory 0.9 (name matching the upcoming bmt version to be released, 0.9.0) and point it to the latest sha commit (2827e2ce4421c5deb032e90702cff509a9540a91) - current released version is 0.8.0

Thus, all the methodology changes would be backward compatible for already existing engagements to which hackers are invited to

@adarshaks91 adarshaks91 changed the title [BC-33602] Point 0.1 version to the old commit sha [BC-33602] Submodule directory versioning fix Sep 12, 2025
@adarshaks91 adarshaks91 marked this pull request as draft September 12, 2025 11:22
@adarshaks91 adarshaks91 requested a review from Copilot September 12, 2025 11:25
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements versioning for the methodology-taxonomy submodule by creating separate version-specific directories and updating the Git submodule configuration.

  • Adds a new submodule directory "0.8" pointing to the latest commit to match current BMT version 0.8.0
  • Updates the existing "0.1" submodule to point to a historical commit before AI testing changes
  • Modifies .gitmodules to include configuration for the new 0.8 submodule

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
.gitmodules Adds new submodule configuration for lib/data/0.8 directory
lib/data/0.1 Updated to point to historical commit f8aa5884da64ffc3d000126dff067cd7abde3b6a
lib/data/0.8 New submodule directory pointing to latest commit 2827e2ce4421c5deb032e90702cff509a9540a91

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <[email protected]>
@adarshaks91 adarshaks91 requested a review from Copilot September 12, 2025 11:26
@adarshaks91 adarshaks91 marked this pull request as ready for review September 12, 2025 11:26
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@adarshaks91 adarshaks91 requested a review from Copilot September 12, 2025 15:22
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@adarshaks91 adarshaks91 merged commit 081f255 into main Sep 15, 2025
5 checks passed
@adarshaks91 adarshaks91 deleted the BC-33602-fix branch September 15, 2025 05:21
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