Skip to content

Use PyPi version of BSyncPy and Update to BuildingSync 2.4#412

Merged
nllong merged 2 commits intodevelop-v2from
chore/411-upgrade-notebook
Nov 4, 2021
Merged

Use PyPi version of BSyncPy and Update to BuildingSync 2.4#412
nllong merged 2 commits intodevelop-v2from
chore/411-upgrade-notebook

Conversation

@nllong
Copy link
Copy Markdown
Member

@nllong nllong commented Nov 1, 2021

Any background context you want to provide?

The Jupyter notebook example was on version 2.3.

What does this PR do?

  • Update to 2.4
  • Update python dependencies in poetry.lock

How should this be manually tested?

  • Run the jupyter notebook

What are the relevant tickets?

#411

Screenshots (if appropriate)

@nllong
Copy link
Copy Markdown
Member Author

nllong commented Nov 1, 2021

DO NOT MERGE until pointing the poetry file to a pypi release of this package.

Comment thread docs/notebooks/bsync_examples/example1.xml
@@ -2,6 +2,7 @@
"cells": [
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

There are many changes to this file, easiest to view in Code/Browser.

@nllong nllong requested a review from macintoshpie November 1, 2021 23:11
Copy link
Copy Markdown
Contributor

@macintoshpie macintoshpie left a comment

Choose a reason for hiding this comment

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

Just ran through the notebook in the browser and it all looks good

@nllong nllong changed the title use github branch of new bsyncpy for testing Use PyPi version of BSyncpy and Update to 2.4 Nov 4, 2021
@nllong nllong changed the title Use PyPi version of BSyncpy and Update to 2.4 Use PyPi version of BSyncpy and Update to BuildingSync 2.4 Nov 4, 2021
@nllong nllong changed the title Use PyPi version of BSyncpy and Update to BuildingSync 2.4 Use PyPi version of BSyncPy and Update to BuildingSync 2.4 Nov 4, 2021
Copy link
Copy Markdown
Member Author

@nllong nllong left a comment

Choose a reason for hiding this comment

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

@macintoshpie -- can you check this again before I merge?

Copy link
Copy Markdown
Contributor

@macintoshpie macintoshpie left a comment

Choose a reason for hiding this comment

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

LGTM!

python = "^3.7.9"
jupyterlab = "^3.0.10"
bsyncpy = {git = "https://github.com/BuildingSync/bsyncpy.git", branch="feat/support-bsync-2.4" }
bsyncpy = "0.2.0"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

😎

@nllong nllong merged commit dbbab69 into develop-v2 Nov 4, 2021
@nllong nllong deleted the chore/411-upgrade-notebook branch November 4, 2021 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants