Skip to content

Enable eln-extensions view - #122

Merged
sgallagher merged 1 commit into
fedora-eln:masterfrom
yselkowitz:master
Aug 4, 2026
Merged

Enable eln-extensions view#122
sgallagher merged 1 commit into
fedora-eln:masterfrom
yselkowitz:master

Conversation

@yselkowitz

@yselkowitz yselkowitz commented Aug 2, 2026

Copy link
Copy Markdown
Member

This was just split out of ELN Extras:

fedora-eln/content-resolver-input#1645
fedora-eln/eln#594

Summary by CodeRabbit

  • Chores
    • Updated the build process to include the eln-extensions label alongside existing labels.

@yselkowitz
yselkowitz requested a review from bhoy-troy August 2, 2026 03:50
@coderabbitai

coderabbitai Bot commented Aug 2, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 08cc27c0-50f7-4c9b-be07-1f9c89878e75

📥 Commits

Reviewing files that changed from the base of the PR and between ef7c73d and 4f3ebca.

📒 Files selected for processing (1)
  • refresh.sh

📝 Walkthrough

Walkthrough

The refresh command now passes the eln-extensions label to the content resolver alongside eln and eln-extras.

Changes

ELN refresh configuration

Layer / File(s) Summary
Content resolver label update
refresh.sh
The content resolver invocation now processes eln-extensions with the existing labels.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: bhoy-troy

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the change that enables the eln-extensions view.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sgallagher sgallagher left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Seems fine, but I wonder what this will do for the runtime.

@yselkowitz

Copy link
Copy Markdown
Member Author

If you mean "run time", not much, given that Extensions is a few dozen packages on x86_64 and only a handful on other arches (ROCm being the difference), and these were already listed in Extras, I'm just splitting them out now to get Extras closer to EPEL (only).

@sgallagher
sgallagher merged commit fb95e5d into fedora-eln:master Aug 4, 2026
3 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.

2 participants