Skip to content

Conversation

@Raubzeug
Copy link
Contributor

@Raubzeug Raubzeug commented Nov 28, 2024

CI Results

Test Status: βœ… PASSED

πŸ“Š Full Report

Total Passed Failed Flaky Skipped
208 208 0 0 0

😟 No changes in tests. πŸ˜•

Bundle Size: βœ…

Current: 65.96 MB | Main: 65.96 MB
Diff: +0.04 KB (0.00%)

βœ… Bundle size unchanged.

ℹ️ CI Information
  • Test recordings for failed tests are available in the full report.
  • Bundle size is measured for the entire 'dist' directory.
  • πŸ“Š indicates links to detailed reports.
  • πŸ”Ί indicates increase, πŸ”½ decrease, and βœ… no change in bundle size.

// TODO: colors used in charts as well, need to move to constants
// 11 distinct colors from https://mokole.com/palette.html
export const COLORS = [
'#008000', // green
Copy link
Collaborator

Choose a reason for hiding this comment

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

Before all the changes, there were 11 colors (see comment in the code above). Let's not change the number of colors.

@Raubzeug Raubzeug requested a review from ValeraS November 28, 2024 09:32
'#008000', // green
'#4169e1', // royalblue
'#ffd700', // gold
'#ff8c00', // darkorange
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why is this deleted?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

unintentionally :(

@Raubzeug Raubzeug requested a review from ValeraS November 28, 2024 11:21
@Raubzeug Raubzeug enabled auto-merge November 28, 2024 11:22
@Raubzeug Raubzeug added this pull request to the merge queue Nov 28, 2024
Merged via the queue into main with commit 97c66b8 Nov 28, 2024
5 checks passed
@Raubzeug Raubzeug deleted the fix-palette branch November 28, 2024 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants