Skip to content

Conversation

@codeZeilen
Copy link
Owner

Depends on #2

@codeZeilen codeZeilen changed the title Sendmail header tests Sendmail header and cc/bcc tests Apr 14, 2025
@codeZeilen codeZeilen requested a review from Copilot April 14, 2025 13:42
Copy link

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 updates documentation and CI workflows to reflect the renaming of the test suite framework from PorTS to RosettaTest, supporting the sendmail header and cc/bcc tests.

  • Updated README.md to replace references and instructions from PorTS to RosettaTest.
  • Modified GitHub workflow files (Ruby, Python, JavaScript, and interpreter tests) to point to the new RosettaTest file paths and commands.

Reviewed Changes

Copilot reviewed 354 out of 369 changed files in this pull request and generated no comments.

Show a summary per file
File Description
README.md Replaced PorTS references with RosettaTest in description and instructions.
.github/workflows/ruby-suites.yml Updated Ruby command paths from ports-rb to rosetta-test-rb.
.github/workflows/python-suites.yml Modified Python test commands and dependencies from ports-py to rosetta-test-py.
.github/workflows/javascript-suites.yml Updated JavaScript command paths from ports-js to rosetta-test-js.
.github/workflows/interpreter-tests.yml Adjusted interpreter test references for Python, Ruby, and JavaScript to RosettaTest.
Files not reviewed (15)
  • .gitattributes: Language not supported
  • .smalltalk.ston: Language not supported
  • ports-s/BaselineOfPortsS/BaselineOfPortsS.class.st: Language not supported
  • ports-s/BaselineOfPortsS/package.st: Language not supported
  • ports-s/PortsS-Core/package.st: Language not supported
  • ports-s/PortsS-Interpreter/Collection.extension.st: Language not supported
  • ports-s/PortsS-Interpreter/False.extension.st: Language not supported
  • ports-s/PortsS-Interpreter/Object.extension.st: Language not supported
  • ports-s/PortsS-Interpreter/PortsSyntaxError.class.st: Language not supported
  • ports-s/PortsS-Interpreter/PortsTypeError.class.st: Language not supported
  • ports-s/PortsS-Interpreter/package.st: Language not supported
  • ports-s/PortsS-Suites/PortsFSSuite.class.st: Language not supported
  • ports-s/PortsS-Suites/PortsJSONRfcSuite.class.st: Language not supported
  • ports-s/PortsS-Suites/package.st: Language not supported
  • ports-s/PortsS-Tests/package.st: Language not supported

@codeZeilen codeZeilen force-pushed the sendmail-header-tests branch 2 times, most recently from 284949a to f519c4e Compare April 15, 2025 09:17
@codeZeilen
Copy link
Owner Author

This branch will not merge easily as it already contains the name change.

@peret
Copy link
Collaborator

peret commented Apr 17, 2025

This branch will not merge easily as it already contains the name change.

Why not merge the name-change PR first and then rebase this branch onto master? :)

@codeZeilen codeZeilen force-pushed the sendmail-header-tests branch from f519c4e to c14ea66 Compare April 17, 2025 08:56
@codeZeilen codeZeilen merged commit 98f4153 into main Apr 17, 2025
4 checks passed
@codeZeilen codeZeilen deleted the sendmail-header-tests branch April 17, 2025 08:57
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