testing dependencies in 3.8

Open

#276 opened on Oct 19, 2024 4 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty: 4/5
Estimated time: 3-5 days
Newbie friendliness: 35/100

Issue type: Feature
Clarity: Mostly clear
Activity status: Stale
Tech stack: github-actions, python
Domain: testing-qa, tooling

Research direction

Start with the linked .github/workflows/on-pull-request.yml lines 18-38 and inspect how dependencies and tests are linked into Packages/*. Account for the Python 3.8 Lib path, the dependency_name/st3/dependency_name layout, and the required .python-version; done means the action supports testing these dependencies under PC 4.

Written by the indexing model from the issue text.

Description

I've been using this action to test lsp_utils dependency in:

https://github.com/sublimelsp/lsp_utils/blob/f2a43fa2db4aeddbcf8ea1ce79ce95f143ddc696/.github/workflows/on-pull-request.yml#L18-L38

With PC 4 now it's no longer enough to link dependency in Packages/*. Now it needs to be linked in ../Lib/python38/*. And not the root directory of the dependency itself has to be linked but rather the dependency_name/st3/dependency_name/ instead.
And the tests I guess still have to be copied/linked to Packages/* with according .python-version added.

I suppose it's a bit of a tricky case to handle now but this is the issue asking for adding support for that. :)

Dominant language: Python
Stars: 113
Forks: 32
PR merge metrics: No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

Read the whole issue, then the project's contributing guide.
Comment on the issue to say you are picking it up — it saves two people doing the same work.
Fork the repository and make your change on a branch.
Open a pull request that references the issue number.