Skip to content

Add RBIs count to snapshot#67

Merged
Morriar merged 5 commits intomasterfrom
at-count-rbis
Apr 30, 2021
Merged

Add RBIs count to snapshot#67
Morriar merged 5 commits intomasterfrom
at-count-rbis

Conversation

@Morriar
Copy link
Contributor

@Morriar Morriar commented Feb 4, 2021

Count & store RBI count in snapshot so we can keep an eye on the amount of files vs. RBI files we have in a project:

image

Signed-off-by: Alexandre Terrasa <alexandre.terrasa@shopify.com>
Signed-off-by: Alexandre Terrasa <alexandre.terrasa@shopify.com>
Signed-off-by: Alexandre Terrasa <alexandre.terrasa@shopify.com>
Signed-off-by: Alexandre Terrasa <alexandre.terrasa@shopify.com>
Signed-off-by: Alexandre Terrasa <alexandre.terrasa@shopify.com>
@Morriar Morriar requested a review from a team February 4, 2021 00:22
desc "snapshot", "Run srb tc and display metrics"
option :save, type: :string, lazy_default: DATA_DIR, desc: "Save snapshot data as json"
option :rbi, type: :boolean, default: true, desc: "Exclude RBI files from metrics"
option :rbi, type: :boolean, default: true, desc: "Include RBI files in metrics"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is the other way around. It comes from a refactor and I forgot to change the description.

pointer-events: none;
}

.area {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Some cleaning in the CSS

JS
end
end

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I really need to find a better way to design/refactor/reuse those components...

Copy link
Contributor

@RyanBrushett RyanBrushett left a comment

Choose a reason for hiding this comment

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

Neat!

@Morriar Morriar merged commit d5b491e into master Apr 30, 2021
@Morriar Morriar deleted the at-count-rbis branch April 30, 2021 21:58
@shopify-shipit shopify-shipit bot temporarily deployed to production May 4, 2021 14:38 Inactive
@Morriar Morriar added the enhancement New feature or request label Mar 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants