Closed NisargChokshi45 closed 1 month ago
Thanks for opening your first pull request in the Marquez project! Please check out our contributing guidelines (https://github.com/MarquezProject/marquez/blob/main/CONTRIBUTING.md).
Name | Link |
---|---|
Latest commit | 6ee3286ffc132b2362ccf3de584c31df287d525f |
Latest deploy log | https://app.netlify.com/sites/peppy-sprite-186812/deploys/66fed97d3b232b0008a018d4 |
Hey @wslulciuc @phixMe, can you please review this PR ?
If possible, can we also add a label named hacktoberfest
in the PR ?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 83.10%. Comparing base (
9e8e3d3
) to head (6ee3286
). Report is 1 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@NisargChokshi45, done! I've also added @phixMe as the reviewer.
Great job! Congrats on your first merged pull request in the Marquez project!
Problem
👋 Thanks for opening a pull request! Please include a brief summary of the problem your change is trying to solve, or bug fix. If your change fixes a bug or you'd like to provide context on why you're making the change, please link the issue as follows:
Closes: #ISSUE-NUMBER
Solution
Please describe your change as it relates to the problem, or bug fix, as well as any dependencies. If your change requires a database schema migration, please describe the schema modification(s) and whether it's a backwards-incompatible or backwards-compatible change.
One-line summary: Updated event count chip rendering
Checklist
CHANGELOG.md
(Depending on the change, this may not be necessary)..sql
database schema migration according to Flyway's naming convention (if relevant)