adoptium / adoptium/aqa-tests

combine VENDOR_TEST_BRANCHES and VENDOR_TEST_SHAS into one param

Open
#5,321 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
HTML
Stars
155
Forks
346
Avg merge
2d 9h
Merged PRs (30d)
36

Description

Currently, for running vendor repo tests, users can provide VENDOR_TEST_BRANCHES or VENDOR_TEST_SHAS. To reduce the parameters that we have, we combine VENDOR_TEST_BRANCHES and VENDOR_TEST_SHAS into one parameter. That is, keep VENDOR_TEST_BRANCHES and allow users to set branches, tags, or SHA into this parameter.

Contributor guide

Open the contributing guide

Research direction

Start by locating all references to VENDOR_TEST_BRANCHES and VENDOR_TEST_SHAS in the vendor repository test configuration and documentation. Determine how branches, tags, and SHAs are parsed today, then update the configuration so VENDOR_TEST_BRANCHES accepts all three forms and remove the separate parameter. Done means vendor tests work with each supported reference and existing tests pass.

Written by the indexing model from the issue text.

Assessment

Domain
testing
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.