Skip to content

Simplify test262-harness.py#4910

Merged
robertsipka merged 1 commit into
jerryscript-project:masterfrom
ossy-szeged:test262-drop-yaml-parser
Nov 25, 2024
Merged

Simplify test262-harness.py#4910
robertsipka merged 1 commit into
jerryscript-project:masterfrom
ossy-szeged:test262-drop-yaml-parser

Conversation

@ossy-szeged

Copy link
Copy Markdown
Contributor

Use regular expressions to parse the YAML headers of the tests
instead of the imported thirdparty YAML parser.

JerryScript-DCO-1.0-Signed-off-by: Csaba Osztrogonác csaba.osztrogonac@h-lab.eu

@ossy-szeged
ossy-szeged force-pushed the test262-drop-yaml-parser branch from fc5e223 to 5a92b62 Compare December 21, 2021 14:39
Comment thread tools/runners/test262-harness.py Outdated
Comment thread tools/runners/test262-harness.py Outdated
@ossy-szeged
ossy-szeged force-pushed the test262-drop-yaml-parser branch 2 times, most recently from cf3a0d6 to e44c600 Compare January 10, 2022 20:48
@ossy-szeged

Copy link
Copy Markdown
Contributor Author

Based on #4959, it should be merged first.

@rtakacs

rtakacs commented Jan 11, 2022

Copy link
Copy Markdown
Contributor

LGTM (informal)

@ossy-szeged
ossy-szeged force-pushed the test262-drop-yaml-parser branch from e44c600 to a587bd8 Compare January 14, 2022 11:01
@LaszloLango LaszloLango added this to the Release 3.0.0 milestone Nov 12, 2024
instead of the imported thirdparty YAML parser.

JerryScript-DCO-1.0-Signed-off-by: Csaba Osztrogonác csaba.osztrogonac@h-lab.eu

@LaszloLango LaszloLango left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@robertsipka robertsipka left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@robertsipka
robertsipka merged commit e4017f0 into jerryscript-project:master Nov 25, 2024
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.

4 participants