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

fix: pre-compile visitable classes regex pattern (#17280) (CP: 24.0) #17282

Merged
merged 1 commit into from
Jul 25, 2023

Conversation

mcollovati
Copy link
Collaborator

Pre compiles the regex pattern to improve build frontend performance during the bytecode scan and collection of reachable classes. Also adds some package exclusions for well-known libraries that should not be visited.

Part of #17234

Pre compiles the regex pattern to improve build frontend performance
during the bytecode scan and collection of reachable classes.
Also adds some package exclusions for well knonw libraries that should
not be visited.

Part of #17234
@sonatype-lift
Copy link

sonatype-lift bot commented Jul 25, 2023

Sonatype Lift is retiring

Sonatype Lift will be retiring on Sep 12, 2023, with its analysis stopping on Aug 12, 2023. We understand that this news may come as a disappointment, and Sonatype is committed to helping you transition off it seamlessly. If you’d like to retain your data, please export your issues from the web console.
We are extremely grateful and thank you for your support over the years.

📖 Read about the impacts and timeline

@sonarcloud
Copy link

sonarcloud bot commented Jul 25, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@github-actions
Copy link

Test Results

   951 files  +19     951 suites  +19   1h 9m 11s ⏱️ + 11m 17s
6 043 tests +24  6 007 ✔️ +23  36 💤 +1  0 ±0 
6 294 runs  +39  6 251 ✔️ +38  43 💤 +1  0 ±0 

Results for commit 8fee0cc. ± Comparison against base commit 2463b85.

@mcollovati mcollovati merged commit be6841b into 24.0 Jul 25, 2023
18 checks passed
@mcollovati mcollovati deleted the cherry/cherrypick-17280-to-24.0 branch July 25, 2023 13:29
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Vaadin 24.0.12.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants