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

Dynamically generate help text + rename test files #311

Merged
merged 8 commits into from
Aug 16, 2020

Conversation

alysivji
Copy link
Member

@alysivji alysivji commented Aug 16, 2020

What does this do

  • generate help text based on what features the workspace has enabled
  • only commands for features that are enabled are active
  • rename test files

Why are we doing this

Improve user experience

How should this be tested

Added tests around this functionality

Migrations

n/a

Dependencies

n/a

Callouts

n/a

@codecov-commenter
Copy link

codecov-commenter commented Aug 16, 2020

Codecov Report

Merging #311 into master will decrease coverage by 0.14%.
The diff coverage is 85.71%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #311      +/-   ##
==========================================
- Coverage   92.41%   92.26%   -0.15%     
==========================================
  Files          70       70              
  Lines        2253     2287      +34     
==========================================
+ Hits         2082     2110      +28     
- Misses        171      177       +6     
Impacted Files Coverage Δ
...usy_beaver/apps/slack_integration/slash_command.py 90.35% <70.00%> (-4.55%) ⬇️
...eaver/apps/slack_integration/event_subscription.py 96.29% <100.00%> (+0.04%) ⬆️
busy_beaver/apps/slack_integration/interactors.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3e9d890...5454f73. Read the comment docs.

@alysivji alysivji merged commit 6b469e6 into busy-beaver-dev:master Aug 16, 2020
@alysivji alysivji deleted the help-text-generator branch August 16, 2020 03:24
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.

2 participants