Skip to content

Fix the Script Process #874

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
5 tasks done
cdavernas opened this issue May 31, 2024 · 6 comments · Fixed by #1063
Closed
5 tasks done

Fix the Script Process #874

cdavernas opened this issue May 31, 2024 · 6 comments · Fixed by #1063
Assignees
Labels
area: ctk Changes in the CTK (Compliance Test Kit) area: spec Changes in the Specification change: documentation Improvements or additions to documentation. It won't impact a version change. change: fix Something isn't working. Impacts in a minor version change. wontfix This will not be worked on
Milestone

Comments

@cdavernas
Copy link
Member

cdavernas commented May 31, 2024

What would you like to be added:

Fix the script process by:

  • Document that runtimes must support js (executed by NodeJS) and python (executed by Python) by default.
  • Document that language should also specify the version (ex: js:22.2.0, python:3).
  • Add an arguments property, of type string[], used to pass arguments to the script to execute. Document that the arguments can contain runtime expressions.
  • Document that environment can contain runtime expressions.
  • Add an object used to define how the output is read from a script's execution (see below example)

Define script output proposal:

run:
  script:
    language: jq:22.2.0
    code: ...
    output:
      console: {} #indicates that we should read the output from the console (stdout)
      file: {} #indicates that the script is expected to output to a file relative to the script's working directory
      code: {} #indicates that the script outputs its exit code

Why is this needed:

Allow proper execution of script processes.

@cdavernas cdavernas added change: fix Something isn't working. Impacts in a minor version change. change: documentation Improvements or additions to documentation. It won't impact a version change. area: spec Changes in the Specification area: ctk Changes in the CTK (Compliance Test Kit) labels May 31, 2024
@cdavernas cdavernas added this to the v1.0.0-alpha2 milestone May 31, 2024
Copy link

github-actions bot commented Aug 4, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Copy link

github-actions bot commented Oct 1, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Copy link

github-actions bot commented Jan 7, 2025

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@cdavernas cdavernas added wontfix This will not be worked on and removed Stale Issue labels Jan 7, 2025
@cdavernas
Copy link
Member Author

As previously discussed with @fjtirado, @ricardozanini, and @JBBianchi multiple times over the past year, we have decided to retain the script process while restricting the versions of the supported languages.

This approach provides the best of both worlds: it preserves cross-compatibility for workflows while allowing users to use alternative language versions by leveraging container processes when necessary.

@cdavernas
Copy link
Member Author

cdavernas commented Jan 10, 2025

Following languages are supported:

Language Version
JavaScript ES2024
Python 3.13.1

@cdavernas cdavernas linked a pull request Jan 10, 2025 that will close this issue
8 tasks
@github-project-automation github-project-automation bot moved this from Backlog to Done in Progress Tracker Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ctk Changes in the CTK (Compliance Test Kit) area: spec Changes in the Specification change: documentation Improvements or additions to documentation. It won't impact a version change. change: fix Something isn't working. Impacts in a minor version change. wontfix This will not be worked on
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant