Skip to content

Make more plugins optionally built - #915

Merged
aaust merged 4 commits into
masterfrom
sdobbs_plugin_sweep_mar2025
Jun 30, 2026
Merged

Make more plugins optionally built#915
aaust merged 4 commits into
masterfrom
sdobbs_plugin_sweep_mar2025

Conversation

@sdobbs

@sdobbs sdobbs commented Mar 26, 2025

Copy link
Copy Markdown
Contributor

Also move older unused plugins to a "retired" folder, so that we don't lose their history

I tried to implement the discussion in issue #843

Maybe the PR can be used to get analyzers to confirm these changes.

Also move older unused plugins to a "retired" folder, so that we don't lose
their history
@sdobbs

sdobbs commented Mar 26, 2025

Copy link
Copy Markdown
Contributor Author

the tests are failing since this PR removes some of the plugins used for these tests... ;_;

@nsjarvis

nsjarvis commented Mar 26, 2025 via email

Copy link
Copy Markdown
Contributor

@sdobbs

sdobbs commented Mar 26, 2025

Copy link
Copy Markdown
Contributor Author

Not a problem - done!

@gluex

gluex commented Mar 27, 2025

Copy link
Copy Markdown

@sdobbs

sdobbs commented Apr 10, 2026

Copy link
Copy Markdown
Contributor Author

Maybe we should think about what to do with this PR...

@gluex

gluex commented Jun 30, 2026

Copy link
Copy Markdown

Test status for this pull request: SUCCESS

Summary: summary.txt
Logs: results/log

Build log: make_sdobbs_plugin_sweep_mar2025.log
Build report: report_sdobbs_plugin_sweep_mar2025.txt

@gluex

gluex commented Jun 30, 2026

Copy link
Copy Markdown

Test status for this pull request: SUCCESS

Summary: summary.txt
Logs: results/log

Build log: make_sdobbs_plugin_sweep_mar2025.log
Build report: report_sdobbs_plugin_sweep_mar2025.txt

@aaust aaust left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks, this reduces the compiled plugins from 171 to 99, which makes the build faster and lighter.

@aaust
aaust merged commit c83557f into master Jun 30, 2026
2 checks passed
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.

4 participants