Skip to content

Conversation

@mrakitin
Copy link
Contributor

@mrakitin mrakitin commented May 3, 2019

A possible fix for #129.

@prjemian
Copy link
Contributor

prjemian commented May 7, 2019

So, are we ready to merge?

* BCDA-APS#131 WIP

* BCDA-APS#131 move definition to package init

* BCDA-APS#131 WIP - problems with historydict

* BCDA-APS#131 as recommended

* BCDA-APS#131 sort (below dev requirements)

* BCDA-APS#131 no need to package the requirements list

* BCDA-APS#131 don't list requirements for run-time use
Copy link
Contributor Author

@mrakitin mrakitin left a comment

Choose a reason for hiding this comment

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

We forgot to implement 1 change regarding the databroker[all] line. Re-iterating here.

@mrakitin
Copy link
Contributor Author

mrakitin commented May 7, 2019

@prjemian, it's ready to go. Thanks for all your contributions via mrakitin#1.

@prjemian prjemian merged commit 8f60dac into BCDA-APS:master May 7, 2019
@mrakitin mrakitin deleted the fix-conda-recipe-and-pip-deps branch May 7, 2019 20:16
@mrakitin
Copy link
Contributor Author

mrakitin commented May 7, 2019

Awesome, thanks!

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