Skip to content

fix: add original license and copyrights for the referenced code#438

Merged
9aoy merged 3 commits intomainfrom
fix-copyright
Aug 4, 2025
Merged

fix: add original license and copyrights for the referenced code#438
9aoy merged 3 commits intomainfrom
fix-copyright

Conversation

@9aoy
Copy link
Copy Markdown
Collaborator

@9aoy 9aoy commented Aug 4, 2025

Summary

Some testing features and utilities in Rstest have been adapted from Vitest, Jest and Playwright. Add original license and copyrights for the referenced code.

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copilot AI review requested due to automatic review settings August 4, 2025 05:51
@netlify
Copy link
Copy Markdown

netlify bot commented Aug 4, 2025

Deploy Preview for rstest-dev ready!

Name Link
🔨 Latest commit 5f801ec
🔍 Latest deploy log https://app.netlify.com/projects/rstest-dev/deploys/689059b70a6f910008858a59
😎 Deploy Preview https://deploy-preview-438--rstest-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@9aoy 9aoy requested review from fi3ework and removed request for Copilot August 4, 2025 05:51
Copilot AI review requested due to automatic review settings August 4, 2025 06:48
Copy link
Copy Markdown
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 adds original license and copyright information for code that has been adapted from Vitest, Jest, and Playwright projects. The purpose is to ensure proper attribution and legal compliance for referenced code.

  • Adds MIT license headers with copyright notices to files containing Vitest-adapted code
  • Adds Jest MIT license header to console.ts file
  • Adds Apache 2.0 license header to fixtures.ts file for Playwright-adapted code
  • Updates the main LICENSE.md file to include comprehensive attribution for all referenced projects
  • Refactors license generation code into a separate module

Reviewed Changes

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

Show a summary per file
File Description
packages/core/src/types/expect.ts Adds MIT license header for Vitest-adapted code
packages/core/src/runtime/worker/console.ts Adds MIT license header for Jest-adapted code
packages/core/src/runtime/runner/fixtures.ts Adds Apache 2.0 license header for Playwright-adapted code
packages/core/src/runtime/api/snapshot.ts Adds MIT license header for Vitest-adapted code
packages/core/src/runtime/api/poll.ts Adds MIT license header for Vitest-adapted code
packages/core/src/runtime/api/expect.ts Adds MIT license header for Vitest-adapted code
packages/core/src/reporter/windowedRenderer.ts Adds MIT license header for Vitest-adapted code
packages/core/src/reporter/summary.ts Adds MIT license header for Vitest-adapted code
packages/core/src/pool/index.ts Adds MIT license header for Vitest-adapted code
packages/core/rslib.config.ts Refactors license plugin code to separate file
packages/core/licensePlugin.ts New file containing license generation logic with comprehensive attribution
packages/core/LICENSE.md Updates main license with comprehensive third-party attribution section

* This method is modified based on source found in
* https://github.com/vitest-dev/vitest/blob/e8ce94cfb5520a8b69f9071cc5638a53129130d6/packages/vitest/src/integrations/chai/poll.ts
*
*
Copy link

Copilot AI Aug 4, 2025

Choose a reason for hiding this comment

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

[nitpick] There's an extra blank line in the comment block. Consider removing line 5 to maintain consistent formatting with other license headers.

Suggested change
*

Copilot uses AI. Check for mistakes.
@9aoy 9aoy merged commit e455ea0 into main Aug 4, 2025
16 checks passed
@9aoy 9aoy deleted the fix-copyright branch August 4, 2025 10:51
@9aoy 9aoy mentioned this pull request Aug 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants