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.

Sign up for GitHub

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

Jump to bottom

Bump Dash version to 2.0 #161

Merged
merged 1 commit into from
Sep 13, 2021
Merged

Bump Dash version to 2.0 #161

merged 1 commit into from
Sep 13, 2021

Conversation

HansKallekleiv
Copy link
Collaborator

  • 📜 I have broken down my PR into the following tasks:
    • Bump Dash dependency to 2.0
    • Simplify import statement as suggested in Dash 2.0 migration
  • 🤖 I have added tests, or extended existing tests, to cover any new features or bugs fixed in this PR.
  • 📖 I have considered adding a new entry in CHANGELOG.md, and added it if should be communicated there.

anders-kiaer
anders-kiaer requested changes Sep 12, 2021
View reviewed changes
Copy link
Collaborator

@anders-kiaer anders-kiaer left a comment

Choose a reason for hiding this comment

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

LGTM! Bump dash version constraint in setup.py to >=2.0?

CHANGELOG.md Outdated Show resolved Hide resolved
@anders-kiaer anders-kiaer added the next release 🚢 To be included in next release label Sep 12, 2021
@anders-kiaer anders-kiaer merged commit 3708bec into equinor:master Sep 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anders-kiaer anders-kiaer anders-kiaer approved these changes

Assignees

@HansKallekleiv HansKallekleiv

Labels
maintenance next release 🚢 To be included in next release
Projects
Webviz
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@HansKallekleiv @anders-kiaer