Skip to content

Comments

perf(linter): reduce the RuleEnum enum size from 168 to 16 bytes#1783

Merged
Boshen merged 2 commits intomainfrom
perf-linter-enum-size
Dec 22, 2023
Merged

perf(linter): reduce the RuleEnum enum size from 168 to 16 bytes#1783
Boshen merged 2 commits intomainfrom
perf-linter-enum-size

Conversation

@Boshen
Copy link
Member

@Boshen Boshen commented Dec 22, 2023

No description provided.

@github-actions github-actions bot added the A-linter Area - Linter label Dec 22, 2023
@codspeed-hq
Copy link

codspeed-hq bot commented Dec 22, 2023

CodSpeed Performance Report

Merging #1783 will improve performances by 14.56%

Comparing perf-linter-enum-size (dde4b70) with main (521aa2c)

Summary

⚡ 3 improvements
✅ 13 untouched benchmarks

Benchmarks breakdown

Benchmark main perf-linter-enum-size Change
linter[antd.js] 10.8 s 9.5 s +14.55%
linter[typescript.js] 19.5 s 17.1 s +14.43%
linter[react.development.js] 135.1 ms 117.9 ms +14.56%

@Boshen Boshen changed the title perf(linter): reduce the RuleEnum enum size from 168 to 16 bytes perf(linter): reduce the RuleEnum enum size from 168 to 16 bytes Dec 22, 2023
@Boshen Boshen force-pushed the perf-linter-enum-size branch from 4e30903 to dde4b70 Compare December 22, 2023 14:40
@Boshen Boshen marked this pull request as ready for review December 22, 2023 14:53
@Boshen Boshen merged commit 51a243b into main Dec 22, 2023
@Boshen Boshen deleted the perf-linter-enum-size branch December 22, 2023 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-linter Area - Linter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants