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

Reduce value set of Analyst Consensus #1291

Merged
merged 2 commits into from
May 14, 2024
Merged

Reduce value set of Analyst Consensus #1291

merged 2 commits into from
May 14, 2024

Conversation

marvinruder
Copy link
Owner

  • Add analystRatings stock property
  • Add Analyst Rating Bar
  • Simplify error handling in fetchers
  • Add helpers for mathematical operations on Records
  • Remove Selenium-related code
  • Run more tests concurrently

@marvinruder
Reduce value set of Analyst Consensus
9b65c75 
* Add `analystRatings` stock property
* Add Analyst Rating Bar
* Simplify error handling in fetchers
* Add helpers for mathematical operations on Records
* Remove Selenium-related code
* Run more tests concurrently

Signed-off-by: Marvin A. Ruder <[email protected]>
@marvinruder marvinruder added feature New feature or request frontend Related to frontend package backend Related to backend package commons Related to commons package data-provider Related to fetching data from providers postgres Related to PostgreSQL database labels May 14, 2024
@marvinruder marvinruder self-assigned this May 14, 2024
@marvinruder marvinruder linked an issue May 14, 2024 that may be closed by this pull request
Reduce value set of Analyst Consensus #1234
Closed
@marvinruder
Fix dependencies
5363569 
Signed-off-by: Marvin A. Ruder <[email protected]>
@codacy-production Codacy Production
Copy link

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
✅ +0.00% (target: +0.00%) ✅ 100.00% (target: 100.00%)
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (760779c) 10781 10781 100.00%
Head commit (5363569) 10869 (+88) 10869 (+88) 100.00% (+0.00%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#1291) 176 176 100.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Codacy will stop sending the deprecated coverage status from June 5th, 2024. Learn more

@marvinruder marvinruder merged commit 16c8ecc into main May 14, 2024
11 checks passed
@marvinruder marvinruder deleted the 1234-reduce-value-set-of-analyst-consensus branch May 14, 2024 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@marvinruder marvinruder

Labels
backend Related to backend package commons Related to commons package data-provider Related to fetching data from providers feature New feature or request frontend Related to frontend package postgres Related to PostgreSQL database
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Reduce value set of Analyst Consensus
1 participant
@marvinruder