Skip to content

Add MIT LICENSE file to repository - #4

Merged
Sun Haoran (haoranpb) merged 2 commits into
mainfrom
copilot/fix-06eb8b4f-b016-46d7-ae90-dbd7cb40eebe
Sep 29, 2025
Merged

Add MIT LICENSE file to repository#4
Sun Haoran (haoranpb) merged 2 commits into
mainfrom
copilot/fix-06eb8b4f-b016-46d7-ae90-dbd7cb40eebe

Conversation

Copilot AI commented Sep 29, 2025

Copy link
Copy Markdown
Contributor

This PR adds the missing MIT LICENSE file to the BC-Bench repository, addressing issue #[issue_number].

Problem

The repository was missing a LICENSE file, which prevents users from understanding how to use the project in a compliant manner. This is a common requirement for open source projects to clarify usage rights and legal obligations.

Solution

Added the standard MIT LICENSE file using the exact content from Microsoft's official repo-templates at:
https://github.com/microsoft/repo-templates/blob/main/shared/LICENSE

The MIT license provides users with clear permissions to:

  • Use the software for any purpose
  • Modify and distribute the software
  • Include the software in commercial products
  • All while maintaining the copyright notice and license terms

Changes

  • Added LICENSE file to the repository root containing the MIT license text
  • Copyright attribution to Microsoft Corporation as per Microsoft's standard template

This change follows Microsoft's open source guidelines and ensures the project meets standard licensing requirements for open source repositories.

Original prompt

This section details on the original issue you should resolve

<issue_title>This repo is missing a LICENSE file</issue_title>
<issue_description>This repository is currently missing a LICENSE file.

A license helps users understand how to use your project in a compliant manner. You can find the standard MIT license Microsoft uses at: https://github.com/microsoft/repo-templates/blob/main/shared/LICENSE.

If you would like to learn more about open source licenses, please visit the document at https://aka.ms/license (Microsoft-internal guidance).
</issue_description>

<agent_instructions>Add the linked license file</agent_instructions>

Comments on the Issue (you are Copilot in this section)

Fixes microsoft/BC-Bench#1

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

Co-authored-by: haoranpb <27280733+haoranpb@users.noreply.github.com>
Copilot AI changed the title [WIP] This repo is missing a LICENSE file Add MIT LICENSE file to repository Sep 29, 2025
@haoranpb
Sun Haoran (haoranpb) marked this pull request as ready for review September 29, 2025 08:05
@haoranpb
Sun Haoran (haoranpb) merged commit 2e386d8 into main Sep 29, 2025
1 check passed
@haoranpb
Sun Haoran (haoranpb) deleted the copilot/fix-06eb8b4f-b016-46d7-ae90-dbd7cb40eebe branch September 29, 2025 08:05
Marko Aleksandric (AleksandricMarko) added a commit that referenced this pull request Jul 31, 2026
…ding

Applies Haoran's review feedback for the extensibility-request categories:

- Rename to explicit `extensibility-request-implement` / `extensibility-request-triage`
  across category ids, enum members, classes, dataset files and module files (#1/#2/#4).
- Slim both prompt templates to reflect real-world usage; category-specific rules now
  live in the ai-ext-fix / argus-triage skills (#3).
- Merge duplicate `bcbench.results` import in commands/evaluate.py (#5).
- Uptake #761: both ext entries subclass `RepoGroundedEntry` (#6/#7).
- Triage `patch` defaults to None; stripped from the dataset (#8).
- Constrain triage label fields with a `ManagedLabel` Literal to catch dataset typos (#9).
- Move the ext dataset classes into dataset/extensibility_request.py, like code review (#10).
- Drop `fetch_commit_if_missing` from both ext pipelines to match the other pipelines (#11).
- Keep ext-implement judge-only for now; container build/publish noted as future work (#12).
- Grade triage with the NL2AL LMChecklist instead of the code-review-style hybrid:
  `ExtRequestTriageResult` removed in favour of `JudgeBasedEvaluationResult`, expected_*
  fields replaced by an `expected` checklist, evaluators `lm_checklist`, core score
  `test_passed`; 17 triage entries migrated (#13/#14/#15/#16).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: fb712366-b026-470c-9d09-1bd9e9f52a31
Wenjie Fan (gggdttt) pushed a commit that referenced this pull request Aug 13, 2026
- Move code-review-only 'articles' off base EntryMetadata into a CodeReviewEntryMetadata subclass (haoran #1)

- Share an ArticleId type between ReviewComment.article and metadata.articles (haoran #3)

- Add a model validator keeping per-comment and entry-level article annotations disjoint so they cannot drift (haoran #2)

- Replace the ad-hoc dataset coverage CLI command with a notebook, matching the bug-fix precedent (haoran #4)
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.

This repo is missing a LICENSE file

2 participants