Skip to content
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

setup: Pin libcove requirement #63

Merged
merged 1 commit into from
Nov 17, 2020
Merged

setup: Pin libcove requirement #63

merged 1 commit into from
Nov 17, 2020

Conversation

Bjwebb
Copy link
Collaborator

@Bjwebb Bjwebb commented Nov 6, 2020

(Due to breaking changes in later versions).

Original issue on this: #61

(Due to breaking changes in later versions).

#61
@Bjwebb
Copy link
Collaborator Author

Bjwebb commented Nov 6, 2020

This is necessary due to breaking changes in lib-cove 0.19.0 and 0.20.0. I started a PR to fix the former, but it has some issues I've not had time to follow up on. The changes for both are fairly minor, I've just not been able to block out time to make, test and review them properly. So, this PR is to pin the requirement for now. I hope to look into making the updates in a few weeks time (although it could be in the new year), as part of our "maintenance" work on BODS tech.

(BTW, I'm off next week, so won't respond to any comments until the 17th).

@Bjwebb Bjwebb merged commit caf2fae into master Nov 17, 2020
@Bjwebb Bjwebb deleted the pin-libcove branch November 17, 2020 11:10
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