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

DPPA-1123 Rename mibitracker-client to mibilib #99

Merged
merged 4 commits into from
Aug 26, 2020
Merged

DPPA-1123 Rename mibitracker-client to mibilib #99

merged 4 commits into from
Aug 26, 2020

Conversation

rfinck
Copy link
Contributor

@rfinck rfinck commented Aug 26, 2020

No description provided.

@rfinck rfinck requested a review from murataksoy August 26, 2020 16:59
channels:
- conda-forge
- defaults
dependencies:
- pip=19.1.1
- python=3.7.3
- pip:
- astroid==2.2.5
Copy link
Contributor

Choose a reason for hiding this comment

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

Why was this change necessary? It seems to conflict with the mibitof version.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

astroid is a dependency of pylint and the version it was pulling caused it to error (see circleci failure). ultimately we should fix it by updating the pylint version but that should be a separate ticket since we may need to adjust settings. pylint-dev/pylint#2388

Copy link
Contributor

@murataksoy murataksoy left a comment

Choose a reason for hiding this comment

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

Let's follow up to make sure we update our internal libraries if necessary.

@rfinck rfinck merged commit 194ab1d into master Aug 26, 2020
@rfinck rfinck deleted the DPPA-1123 branch August 26, 2020 20:07
rfinck pushed a commit that referenced this pull request Aug 26, 2020
* mibitracker-client -> mibilib

* Update notebook urls.

* Fix pylint
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