Skip to content

Conversation

@Jay07GIT
Copy link
Member

@Jay07GIT Jay07GIT commented Dec 21, 2025

Changes in the PR.
Removed legacy Python-2 compatibility, upgrading AWS dependencies(boto to boto 3), and introducing GitHub Actions for run tests and code coverage.

  1. Removed all Python 2 compatibility code (future, future.moves.*, boto)
  2. Standardized on Python ≥ 3.12

To test the PR-
Run -> tox -e check,py3

To test the github actions -
Run -> act

To test the PR with vinyldns API
To import the PR Branch use this in the requirement.txt
git+https://github.com/Jay07Git/vinyldns-python.git@python3_migration#egg=vinyldns-python

Create a new python file and use a below code to test- List zones from vinyldns

from vinyldns.zone import ListZonesResponse
from vinyldns.client import VinylDNSClient 
def main():
    print(("List Zones"))
    client = VinylDNSClient("https://vinyldns-api-url","AccessKey","SecretKey")

    zones_response = client.list_zones()
    print("Zones count:", len(zones_response.zones))
    for zone in zones_response.zones:
        print("Zone ID:", zone.id)
        print("Zone Name:", zone.name)
 
if __name__ == "__main__":
    main() 

Jay07GIT and others added 5 commits December 21, 2025 19:21
Signed-off-by: Jay07GIT <jeyraj931@gmail.com>
Signed-off-by: Jay07GIT <jeyraj931@gmail.com>
Signed-off-by: Jay07GIT <jeyraj931@gmail.com>
Signed-off-by: Jay07GIT <jeyraj931@gmail.com>
@Jay07GIT Jay07GIT self-assigned this Dec 22, 2025
@Jay07GIT Jay07GIT added kind/maintenance Code cleanup, refactoring, tech debt, etc area/dependencies This issue/PR affects dependencies area/core This issue/PR affects the core functionality labels Dec 22, 2025
Signed-off-by: Jay07GIT <jeyraj931@gmail.com>
Signed-off-by: Jay07GIT <jeyraj931@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/core This issue/PR affects the core functionality area/dependencies This issue/PR affects dependencies kind/maintenance Code cleanup, refactoring, tech debt, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants