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

Roadmap #9

Open
8 of 14 tasks
ChristopherPHolder opened this issue Nov 3, 2022 · 6 comments
Open
8 of 14 tasks

Roadmap #9

ChristopherPHolder opened this issue Nov 3, 2022 · 6 comments
Labels
documentation Improvements or additions to documentation feature extending existing functionality

Comments

@ChristopherPHolder
Copy link
Collaborator

ChristopherPHolder commented Nov 3, 2022

Motivation

Every PR could potentially include a performance regression.
Therefore need a way to continuously measure the performance to inform developers about performance slowdown before the PR gets merged.

To accomplish this, we need a way to compare the measures of the current branch against the measure of the target branch.

Suggested solution

The solution will use @push-based/user-flow and wrap it in a GitHub action to interact with the PR.

There are different building blocks necessary to implement this feature:

Legend:

Building blocks

  • report visualization - a comment in the PR that a human can interpret e.g. MD table
  • report comparison - logic that compares 2 measures and calculates the diff
  • assertions - a way to let the gh action fail if the budget is exceeded
    • performance budgets - A definition of a data structure that can be reflected as JSON

Implementation

We have to split into 2 responsibilities/packages here:

  • 💻 the CLI - @push-based/user-flow
  • ⚙ the CI Action - CI related code e.g. GitHub Action @push-based/user-flow-gh-action

To implement the building blocks we need to solve a couple of core problems:

Documentation

References

@BioPhoton
Copy link
Contributor

BioPhoton commented Nov 3, 2022

[UserFlowName] - Report Visualization

Date/Time: 2022-11-03 15:06

Report Summary
Step Name Gather Mode Performance Accessibility Best Practices Seo Pwa
Navigate to coffee cart navigation 90 97 100 83 30
Select coffee timespan 11/11 - 7/7 - -
Coffee selected snapshot Ø 3/3 15/16 5/5 7/9 -
Checkout order timespan 13/13 - 7/7 - -
Order checked out snapshot Ø 3/3 17/18 5/5 7/9 -
Submit order timespan 13/13 - 7/7 - -
Order submitted snapshot Ø 3/3 15/16 5/5 7/9 -
Settings table here

@BioPhoton
Copy link
Contributor

BioPhoton commented Nov 3, 2022

[UserFlowName] - Report Comparison Visualization

before 80 now 90

Date/Time: 2022-11-03 15:06

Comparison Summary
Step Name Gather Mode Performance Accessibility Best Practices Seo Pwa
Navigate to coffee cart navigation 90 (10⏫) 97 100 83 30
Select coffee timespan 10/11 (1⏬) - 7/7 - -
Coffee selected snapshot Ø 3/3 16/16 (1⏫) 5/5 7/9 -
Checkout order timespan 13/13 - 5/7 (2⏬) - -
Order checked out snapshot Ø 3/3 17/18 5/5 7/9 -
Submit order timespan 13/13 - 6/7 (1⏬) - -
Order submitted snapshot Ø 3/3 15/16 5/5 7/9 -
Settings table here

@BioPhoton
Copy link
Contributor

[UserFlowName] - Report Comparison Visualization v2

before 80 now 90

Date/Time: 2022-11-03 15:06

Comparison Summary
Step Name Gather Mode Performance Accessibility Best Practices Seo Pwa
Navigate to coffee cart navigation 90 (10💚) 97 100 83 30
Select coffee timespan 10/11 (1🔻) - 7/7 - -
Coffee selected snapshot Ø 3/3 16/16 (1💚) 5/5 7/9 -
Checkout order timespan 13/13 - 5/7 (2🔻) - -
Order checked out snapshot Ø 3/3 17/18 5/5 7/9 -
Submit order timespan 13/13 - 6/7 (1🔻) - -
Order submitted snapshot Ø 3/3 15/16 5/5 7/9 -
Settings table here

@BioPhoton
Copy link
Contributor

For the following tasks we need to do more research:

  • handling git checkout and target branch => maybe better in the CLI?

@BioPhoton BioPhoton added documentation Improvements or additions to documentation feature extending existing functionality labels Nov 7, 2022
@BioPhoton
Copy link
Contributor

BioPhoton commented Nov 7, 2022

@ChristopherPHolder
Copy link
Collaborator Author

ChristopherPHolder commented Nov 9, 2022

Related to: push-based/user-flow#94

@BioPhoton BioPhoton changed the title Run User Flows as a Github Action Roadmap Feb 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation feature extending existing functionality
Projects
None yet
Development

No branches or pull requests

2 participants