Skip to content

chore: remove commented out old integration tests, support ref-docs#1912

Merged
filmaj merged 1 commit intomainfrom
cleanup
Sep 3, 2024
Merged

chore: remove commented out old integration tests, support ref-docs#1912
filmaj merged 1 commit intomainfrom
cleanup

Conversation

@filmaj
Copy link
Copy Markdown
Contributor

@filmaj filmaj commented Sep 3, 2024

also removes empty ./integration-test folder, update maintainers guide re: integration tests.

@filmaj filmaj added the docs M-T: Documentation work only label Sep 3, 2024
@filmaj filmaj requested a review from a team September 3, 2024 15:29
@filmaj filmaj self-assigned this Sep 3, 2024
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.26%. Comparing base (7bf89e6) to head (19ac0d0).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1912   +/-   ##
=======================================
  Coverage   90.26%   90.26%           
=======================================
  Files          34       34           
  Lines        7666     7666           
  Branches      381      381           
=======================================
  Hits         6920     6920           
  Misses        734      734           
  Partials       12       12           
Flag Coverage Δ
cli-hooks 94.94% <ø> (ø)
cli-test 96.93% <ø> (ø)
oauth 76.53% <ø> (ø)
socket-mode 59.59% <ø> (ø)
web-api 96.57% <ø> (ø)
webhook 95.27% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

…rent integration test folder, remove old support ref-docs, update maintainers guide re: integration tests.
@filmaj filmaj merged commit 8ba12a9 into main Sep 3, 2024
@filmaj filmaj deleted the cleanup branch September 3, 2024 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs M-T: Documentation work only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants