Skip to content

Conversation

@arrdem
Copy link
Collaborator

@arrdem arrdem commented May 6, 2025


Changes are visible to end-users: yes/no

  • Searched for relevant documentation and updated as needed: yes/no
  • Breaking change (forces users to change their own code or config): yes/no
  • Suggested release notes appear below: yes/no

Test plan

  • Covered by existing test cases
  • New test cases added
  • Manual testing; please provide instructions so we can reproduce:

@github-actions
Copy link

github-actions bot commented May 6, 2025

e2e/use_release folder: LCOV of commit ed053de during CI #1740

Summary coverage rate:
  lines......: 100.0% (2 of 2 lines)
  functions..: 100.0% (1 of 1 function)
  branches...: no data found

Files changed coverage rate: n/a

@arrdem arrdem marked this pull request as ready for review May 29, 2025 19:18
Copy link
Member

@alexeagle alexeagle left a comment

Choose a reason for hiding this comment

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

I'm worried that it's slow, but useful experiment anyway, let's run this way for a while and see what our "NPS" is

@arrdem arrdem merged commit 5dae011 into main May 29, 2025
16 checks passed
@arrdem arrdem deleted the arrdem/automate_docs branch May 29, 2025 20:27
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