Skip to content

chore: Remove setup.py #1218

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.

Sign up for GitHub

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

Merged
KazuCocoa merged 10 commits into from
Apr 19, 2026
+4 −59
Merged

chore: Remove setup.py #1218

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
705967f
chore: remove setup.py and bump major version to v6.0.0
Dor-bl Apr 7, 2026
84a871c
chore: update appium-python-client version to 6.0.0
Dor-bl Apr 17, 2026
b8f443b
chore: update CHANGELOG for v6.0.0 release, removing setup.py details
Dor-bl Apr 17, 2026
261dfc0
refactor: update version retrieval to use importlib.metadata
Dor-bl Apr 17, 2026
94b11a9
chore: revert version to 5.3.0 in pyproject.toml and uv.lock
Dor-bl Apr 18, 2026
5456ae5
refactor: simplify version retrieval by using importlib.metadata dire…
Dor-bl Apr 18, 2026
b6a2fad
chore: update installation instructions to use pip instead of setup.py
Dor-bl Apr 18, 2026
1a42805
refactor: remove unused PackageNotFoundError import from version.py
Dor-bl Apr 18, 2026
7a459ab
refactor: remove unnecessary blank line in version.py
Dor-bl Apr 18, 2026
902e116
chore: remove appium/version.py from version_toml in pyproject.toml
Dor-bl Apr 19, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,15 +26,15 @@ download and unarchive the source tarball (Appium-Python-Client-X.X.tar.gz).
```shell
tar -xvf Appium-Python-Client-X.X.tar.gz
cd Appium-Python-Client-X.X
python setup.py install
pip install .
```

3. Install from source via [GitHub](https://github.com/appium/python-client).

```shell
git clone git@github.com:appium/python-client.git
cd python-client
python setup.py install
pip install .
```

## Compatibility Matrix
Expand Down
9 changes: 2 additions & 7 deletions appium/version.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,11 +12,6 @@
# See the License for the specific language governing permissions and
# limitations under the License.

from importlib import metadata
from importlib.metadata import version as _metadata_version


def _get_version():
return metadata.version('Appium-Python-Client')


version = _get_version()
version = _metadata_version('Appium-Python-Client')
Comment on lines +15 to +17
Copy link

Copilot AI Apr 19, 2026

Choose a reason for hiding this comment

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

version = _metadata_version('Appium-Python-Client') will raise importlib.metadata.PackageNotFoundError when the distribution metadata is not available (e.g., running from a source checkout without an installed wheel/sdist), which makes importing appium.version fail. Wrap the metadata lookup in a try/except and fall back to a safe value (the PR description mentions returning 'unknown').

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Contributor

@mykola-mokhnach mykola-mokhnach Apr 19, 2026

Choose a reason for hiding this comment

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

the PR description needs to be updated

Copy link
Copy Markdown
Contributor Author

@Dor-bl Dor-bl Apr 19, 2026

Choose a reason for hiding this comment

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

Fixed

Comment on lines +15 to +17
Copy link

Copilot AI Apr 19, 2026

Choose a reason for hiding this comment

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

The PR description says _get_version() was improved to handle missing package metadata and return 'unknown', but _get_version() was removed and the module now unconditionally queries metadata at import time. Either reintroduce a helper with the documented fallback behavior or adjust the PR description to match the implemented behavior.

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Contributor

@mykola-mokhnach mykola-mokhnach Apr 19, 2026

Choose a reason for hiding this comment

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

same as above

Copy link
Copy Markdown
Contributor Author

@Dor-bl Dor-bl Apr 19, 2026

Choose a reason for hiding this comment

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

Fixed

5 changes: 0 additions & 5 deletions pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,11 +64,6 @@ dev = [
requires = ["hatchling"]
build-backend = "hatchling.build"

[tool.hatch.version]
source = "regex"
path = "appium/version.py"
pattern = "(?P<version>\\d+\\.\\d+\\.\\d+)"

[tool.hatch.build]
exclude = [
"test/",
Expand Down
45 changes: 0 additions & 45 deletions setup.py
View file
Open in desktop

This file was deleted.

Loading