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

Add endpoint for fetching url link #728

Merged
merged 1 commit into from
Jul 28, 2023
Merged

Conversation

elboy3
Copy link
Contributor

@elboy3 elboy3 commented Jul 27, 2023

@elboy3 elboy3 requested review from a team and dcaustin33 as code owners July 27, 2023 23:57
Copy link
Member

@setu4993 setu4993 left a comment

Choose a reason for hiding this comment

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

💯

@codecov-commenter
Copy link

codecov-commenter commented Jul 28, 2023

Codecov Report

Merging #728 (fa764b9) into main (b78fc3d) will decrease coverage by 0.06%.
The diff coverage is 44.44%.

@@            Coverage Diff             @@
##             main     #728      +/-   ##
==========================================
- Coverage   89.81%   89.75%   -0.06%     
==========================================
  Files         168      168              
  Lines       13592    13601       +9     
==========================================
  Hits        12208    12208              
- Misses       1384     1393       +9     
Files Changed Coverage Δ
dataquality/clients/api.py 51.02% <20.00%> (-0.46%) ⬇️
dataquality/core/log.py 94.92% <66.66%> (-0.63%) ⬇️
dataquality/schemas/route.py 100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@elboy3 elboy3 merged commit 6e303bc into main Jul 28, 2023
@elboy3 elboy3 deleted the chore/add-endpoint-for-run-link branch July 28, 2023 12:45
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.

3 participants