Skip to content

Commit

Permalink
Python upgrade to 3.9 (#113)
Browse files Browse the repository at this point in the history
* running tap-xero on python3.9 and singer-python 5.13.0

* pylint fix

* changes in config.yml

* setup.py and CHANGELOG.md changes
  • Loading branch information
rdeshmukh15 authored Feb 6, 2024
1 parent 844b410 commit e2536cd
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 13 deletions.
15 changes: 5 additions & 10 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ version: 2
jobs:
build:
docker:
- image: 218546966473.dkr.ecr.us-east-1.amazonaws.com/circle-ci:tap-tester-v4
- image: 218546966473.dkr.ecr.us-east-1.amazonaws.com/circle-ci:stitch-tap-tester
steps:
- checkout
- run:
Expand All @@ -21,7 +21,7 @@ jobs:
name: 'pylint'
command: |
source /usr/local/share/virtualenvs/tap-xero/bin/activate
pylint tap_xero --disable 'consider-using-f-string,broad-except,chained-comparison,empty-docstring,fixme,invalid-name,line-too-long,missing-class-docstring,missing-function-docstring,missing-module-docstring,no-else-raise,no-else-return,too-few-public-methods,too-many-arguments,too-many-branches,too-many-lines,too-many-locals,ungrouped-imports,wrong-spelling-in-comment,wrong-spelling-in-docstring,bad-whitespace,unspecified-encoding'
pylint tap_xero --disable 'consider-using-f-string,broad-except,chained-comparison,empty-docstring,fixme,invalid-name,line-too-long,missing-class-docstring,missing-function-docstring,missing-module-docstring,no-else-raise,no-else-return,too-few-public-methods,too-many-arguments,too-many-branches,too-many-lines,too-many-locals,ungrouped-imports,wrong-spelling-in-comment,wrong-spelling-in-docstring,bad-whitespace,unspecified-encoding,broad-exception-raised,use-dict-literal'
- run:
name: 'Unit Tests'
command: |
Expand All @@ -33,14 +33,9 @@ jobs:
aws s3 cp s3://com-stitchdata-dev-deployment-assets/environments/tap-tester/sandbox dev_env.sh
source dev_env.sh
source /usr/local/share/virtualenvs/tap-tester/bin/activate
run-test --tap=tap-xero \
--target=target-stitch \
--orchestrator=stitch-orchestrator \
[email protected] \
--password=$SANDBOX_PASSWORD \
--client-id=50 \
--token=$STITCH_API_TOKEN \
tests
run-test --tap=tap-xero tests
workflows:
version: 2
commit:
Expand Down
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
# Changelog

## 2.3.0
* Updates to run on python 3.9.6 [#113](https://github.com/singer-io/tap-xero/pull/113)

## 2.2.3
* Adds a workaround for a Xero bug to allow pagination to function properly in the `manual_journals` stream [#104](https://github.com/singer-io/tap-xero/pull/104)

Expand Down
6 changes: 3 additions & 3 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,15 +2,15 @@
from setuptools import setup, find_packages

setup(name="tap-xero",
version="2.2.3",
version="2.3.0",
description="Singer.io tap for extracting data from the Xero API",
author="Stitch",
url="http://singer.io",
classifiers=["Programming Language :: Python :: 3 :: Only"],
py_modules=["tap_xero"],
install_requires=[
"singer-python==5.9.0",
"requests==2.20.0",
"singer-python==5.13.0",
"requests==2.31.0",
],
extras_require={
'dev': [
Expand Down

0 comments on commit e2536cd

Please sign in to comment.