Skip to content
This repository has been archived by the owner on Mar 18, 2024. It is now read-only.

chore: update reviewpad to use the summarize marker #1319

Merged
merged 2 commits into from
Jun 2, 2023

Conversation

marcelosousa
Copy link
Contributor

@marcelosousa marcelosousa commented May 26, 2023

Summary generated by Reviewpad on 26 May 23 17:04 UTC

This pull request includes two patches:

  1. Deletes the 'summarize' workflow in reviewpad.yml;
  2. Changes the pull_request_template.md to add the 'reviewpad:summary' tag.

Checklist

All items have to be completed before a PR is merged

  • Adhere to Contribution Guidelines
  • Updates to Decision Records considered?
  • Updates to documentation at DX@Scale Guide considered?
  • Tested changes?
  • Unit Tests new and existing passing locally?

@reviewpad
Copy link

reviewpad bot commented May 26, 2023

Reviewpad Report (Reviewpad ran in dry-run mode because configuration has changed)

📜 Executed actions

$assignCodeAuthorReviewers(1, [], 0)
$commentOnce($sprintf("Thank you @%s for this first contribution!", [$author()]))
$commitLint()
$titleLint()
$warn("Please link an issue to the pull request")
$removeLabels(["medium", "large"])
$addLabel("small")
$addLabel("waiting-for-review")

@codecov
Copy link

codecov bot commented May 29, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (4e8a337) 60.68% compared to head (fe10e55) 60.68%.

❗ Current head fe10e55 differs from pull request most recent head e2d2248. Consider uploading reports for the commit e2d2248 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1319   +/-   ##
=======================================
  Coverage   60.68%   60.68%           
=======================================
  Files          62       62           
  Lines        2147     2147           
  Branches      435      435           
=======================================
  Hits         1303     1303           
  Misses        841      841           
  Partials        3        3           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@azlam-abdulsalam azlam-abdulsalam merged commit 41b7186 into dxatscale:main Jun 2, 2023
4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants