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

Resolve dependency issues #1640

Merged
merged 6 commits into from
Apr 6, 2022
Merged

Resolve dependency issues #1640

merged 6 commits into from
Apr 6, 2022

Conversation

piiq
Copy link
Contributor

@piiq piiq commented Apr 5, 2022 •
edited
Loading

@piiq piiq added bug Fix bug dependencies Work related to dependencies labels Apr 5, 2022
@piiq piiq requested review from Chavithra and LBolte29 April 5, 2022 16:48
@piiq piiq mentioned this pull request Apr 5, 2022
Closed
@piiq piiq changed the title Bump finvizfinance and other deps Resolve dependency issues Apr 5, 2022
Chavithra
Chavithra approved these changes Apr 6, 2022
View reviewed changes
Copy link
Contributor

@Chavithra Chavithra left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@piiq piiq merged commit fa9405e into main Apr 6, 2022
@piiq piiq deleted the update-deps branch April 6, 2022 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Chavithra Chavithra Chavithra approved these changes

@LBolte29 LBolte29 Awaiting requested review from LBolte29

Assignees
No one assigned
Labels
bug Fix bug dependencies Work related to dependencies
Projects
None yet
Milestone
No milestone
2 participants
@piiq @Chavithra