-
Notifications
You must be signed in to change notification settings - Fork 49
Update/automatically #22
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.
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
Changes from all commits
4117d41
56b2bf9
240c0cc
3adc58e
db68ec8
6166645
05e6609
5776d26
971d1f0
22517db
5dcae6b
e540498
5359679
a46ba28
1310399
ce51d66
e29a3b4
880ab7a
ec18969
9eb1fc2
3c75233
4974401
ac5a2f6
5204581
4161f46
71ab237
3f9b979
9f8741b
8524b43
1476914
c1ef040
f8df187
979c896
e179eee
841ae63
ffb1d34
a03d7ca
4542ef8
62c8712
ce2b4b1
d59c376
364ac6d
1f29d88
29888ad
4d86b83
ae45d22
bc17a23
6353bbc
70fda5d
b5ac2a9
4b34279
9577591
c4b058e
fb04cf9
ebf259e
b9b65f2
77d9ed5
5262d7a
1384395
a726956
0ee27ee
5b58522
24eed7b
9342567
8b867be
d5f9c73
7ef1ea8
af8ff10
c545003
65ac763
77d78ad
22c59d0
c81b8a3
b031030
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -19,7 +19,7 @@ jobs: | |
| - checkout # check out source code to working directory | ||
| - run: sudo chown -R circleci:circleci /usr/local/bin | ||
| - restore_cache: | ||
| # Read about caching dependencies: https://circleci.com/docs/2.0/caching/ | ||
| # Read about caching dependencies: https://circleci.com/docs/2.0/caching/ | ||
| key: deps9-{{ .Branch }}-{{ checksum "Pipfile.lock" }} | ||
| - run: | ||
| command: | | ||
|
|
@@ -45,8 +45,8 @@ jobs: | |
| - restore_cache: | ||
| key: deps9-{{ .Branch }}-{{ checksum "Pipfile.lock" }} | ||
| - run: | ||
| command: | | ||
| make setup | ||
| command: | | ||
| make setup | ||
| - save_cache: | ||
| key: deps9-{{ .Branch }}-{{ checksum "Pipfile.lock" }} | ||
| paths: | ||
|
|
@@ -56,6 +56,55 @@ jobs: | |
| git config --local user.email "sysadmin@binary.com" | ||
| git config --local user.name "gh-pages deploy bot" | ||
| make gh-pages | ||
| update_schema: | ||
| <<: *default | ||
| steps: | ||
| - add_ssh_keys: | ||
| fingerprints: | ||
| - "c8:f7:fc:a0:0d:2c:43:93:e3:c7:b6:cf:16:93:98:e1" | ||
| - checkout | ||
| - run: | ||
| name: config git | ||
| command: | | ||
| git config --global user.email "nobody@deriv.com" | ||
| git config --global user.name "Nobody" | ||
| - run: | ||
| name: update schema | ||
| command: | | ||
| git clone https://github.com/binary-com/deriv-developers-portal.git /tmp/deriv-developers-portal | ||
| curl -L https://cpanmin.us | perl - --sudo App::cpanminus | ||
| sudo cpanm -n Dir::Self File::Basename JSON::MaybeXS Log::Any Path::Tiny Template Syntax::Keyword::Try | ||
| BINARYCOM_API_SCHEMA_PATH=/tmp/deriv-developers-portal/config/v3 perl scripts/regen-py.pl | ||
| if [[ $(git diff --shortstat) == ' 2 files changed, 2 insertions(+), 2 deletions(-)' ]] | ||
| then | ||
| echo 'Schema no change' | ||
| exit 0 | ||
| fi | ||
| echo "Schama updated" | ||
| pip3 install bump | ||
| NEXT_VER=$(bump) | ||
| sed -i '/# Changelog/{s/$/\n\n## NEXTVER\n\nSync API/}' CHANGELOG.md | ||
| sed -i "s/NEXTVER/$NEXT_VER/g" CHANGELOG.md | ||
| git add . | ||
| git commit -m 'update schema automatically' | ||
| git push origin HEAD:master | ||
| release: | ||
| <<: *default | ||
| steps: | ||
| - checkout | ||
| - run: | ||
| name: setup pypi | ||
| command: | | ||
| echo "[pypi]" >> ~/.pypirc | ||
| echo "username=__token__" >> ~/.pypirc | ||
| echo "password=$PYPI_TOKEN" >> ~/.pypirc | ||
| - run: | ||
| name: release | ||
| command: | | ||
| python3 -m pip install --upgrade twine | ||
| make build | ||
| python3 -m twine upload --repository pypi dist/* | ||
| echo "deployed to pypi" | ||
| workflows: | ||
| build: | ||
| jobs: | ||
|
|
@@ -78,7 +127,26 @@ workflows: | |
| - "3.10.4" | ||
| - "3.10.10" | ||
| - docs-build-deploy: | ||
| requires: | ||
| - release | ||
| filters: | ||
| branches: | ||
| only: | ||
| - master | ||
| - release: | ||
| requires: | ||
| - test | ||
| filters: | ||
| branches: | ||
| only: | ||
| - master | ||
| update_schema_flow: | ||
| jobs: | ||
| - update_schema | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Since the update_schema is run daily. Are we configuring the trigger via the UI ? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. we needn't trigger via UI, this There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Previously since the trigger was missing i assumed you intended to add the trigger via the project settings rather than specifying the trigger in the yaml itself since as per the docs setting the trigger via the project settings seems to be the preferred approach. https://circleci.com/docs/migrate-scheduled-workflows-to-scheduled-pipelines/ |
||
| triggers: | ||
| - schedule: | ||
| cron: "0 0 * * *" | ||
| filters: | ||
| branches: | ||
| only: | ||
| - master | ||
| - master | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,3 +8,4 @@ venv/ | |
| dist | ||
| .coverage | ||
| build | ||
| tags | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,19 +1,27 @@ | ||
| # Changelog | ||
|
|
||
| ## 0.1.5 | ||
|
|
||
| Change default ws server | ||
|
|
||
| ## 0.1.4 | ||
|
|
||
| Sync API | ||
|
|
||
| ## 0.1.3 | ||
|
|
||
| Fix a typo, which cause ws connection no response | ||
|
|
||
| ## 0.1.2 | ||
|
|
||
| Added middleware support | ||
|
|
||
| ## 0.1.1 | ||
|
|
||
| ### Fixed: | ||
| Fixed a PyPI constraint where the package can only be installed on python ==3.9.6 | ||
|
|
||
| Fixed a PyPI constraint where the package can only be installed on python ==3.9.6 | ||
|
|
||
| ## 0.1.0 | ||
|
|
||
| Initial version. | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does it make sense to have a cpanfile for installing these dependencies ? Or is the dependencies too little to have a separate file for it ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a python project. I'm installing dependencies for that script
scripts/regen-py.pl. I don't think its necessary to maintain a cpanfile.