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

Share an experiment from command palette #2221

Merged
merged 3 commits into from
Aug 19, 2022
Merged

Conversation

mattseddon
Copy link
Member

@mattseddon mattseddon commented Aug 19, 2022

2/2 main <- #2219 <- this

This PR builds on #2219 by adding a command to the palette which can be used to share an experiment.

Demo

Screen.Recording.mov

@mattseddon mattseddon added the product PR that affects product label Aug 19, 2022
@mattseddon mattseddon self-assigned this Aug 19, 2022
)
)

internalCommands.registerExternalCliCommand(
Copy link

Choose a reason for hiding this comment

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

Similar blocks of code found in 2 locations. Consider refactoring.

@mattseddon mattseddon changed the base branch from main to share-experiment August 19, 2022 07:10
@mattseddon mattseddon force-pushed the share-experiment-command branch from 3ace691 to 62ef625 Compare August 19, 2022 07:28
@mattseddon mattseddon marked this pull request as ready for review August 19, 2022 07:29
Base automatically changed from share-experiment to main August 19, 2022 20:53
@mattseddon mattseddon force-pushed the share-experiment-command branch from 62ef625 to b105672 Compare August 19, 2022 20:55
@mattseddon mattseddon enabled auto-merge (squash) August 19, 2022 20:56
@codeclimate
Copy link

codeclimate bot commented Aug 19, 2022

Code Climate has analyzed commit b105672 and detected 2 issues on this pull request.

Here's the issue category breakdown:

Category Count
Duplication 2

The test coverage on the diff in this pull request is 100.0% (85% is the threshold).

This pull request will bring the total coverage in the repository to 96.6% (0.0% change).

View more on Code Climate.

@mattseddon mattseddon merged commit a5b3082 into main Aug 19, 2022
@mattseddon mattseddon deleted the share-experiment-command branch August 19, 2022 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
product PR that affects product
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants