Skip to content

Comments

[#1559] Added DrupalSouth 2025 presentation link to README and docs introduction.#2297

Merged
AlexSkrypnyk merged 1 commit intomainfrom
feature/1559-preso
Feb 17, 2026
Merged

[#1559] Added DrupalSouth 2025 presentation link to README and docs introduction.#2297
AlexSkrypnyk merged 1 commit intomainfrom
feature/1559-preso

Conversation

@AlexSkrypnyk
Copy link
Member

@AlexSkrypnyk AlexSkrypnyk commented Feb 17, 2026

Closes #1559

Summary by CodeRabbit

  • Documentation
    • Added information about the DrupalSouth 2025 presentation with links to presentation materials
    • Enhanced upgrade guidance with additional risk awareness considerations
    • Made minor text corrections and formatting improvements

@coderabbitai
Copy link

coderabbitai bot commented Feb 17, 2026

No actionable comments were generated in the recent review. 🎉


Walkthrough

Documentation updates adding DrupalSouth 2025 presentation information, including promotional content with links to Google Slides and PDF, plus text normalization and extended guidance on risk-aware practices across two README files.

Changes

Cohort / File(s) Summary
Documentation Updates
.vortex/docs/content/README.mdx, README.md
Added DrupalSouth 2025 presentation links (Google Slides and PDF), fixed apostrophe normalization, extended closing paragraph with risk-aware upgrade guidance, and added promotional lead-in content.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

Poem

🐰 A presentation takes flight,
DrupalSouth slides shining bright,
PDFs linked with care so true,
Docs now know just what to do! 📊✨

🚥 Pre-merge checks | ✅ 4 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning The PR adds presentation links to README and docs, but does not fully convert the presentation to a static asset as required by issue #1559. Convert the DrupalSouth 2025 presentation to a static format (HTML/PDF) and include it as a repository asset, not just link to external Google Slides.
Out of Scope Changes check ⚠️ Warning Changes include minor text fixes and extended closing paragraphs beyond the scope of linking the DrupalSouth 2025 presentation. Remove unrelated changes like apostrophe normalization and extended closing paragraph content to keep the PR focused on issue #1559.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main changes: adding DrupalSouth 2025 presentation links to README and docs introduction files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/1559-preso

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

@AlexSkrypnyk AlexSkrypnyk added this to the 1.36.0 milestone Feb 17, 2026
@AlexSkrypnyk
Copy link
Member Author

Code Coverage Report:
  2026-02-17 05:21:48

 Summary:
  Classes:  0.00% (0/1)
  Methods:  0.00% (0/2)
  Lines:   94.12% (176/187)

@github-actions
Copy link

Code Coverage Report:
  2026-02-17 05:21:56

 Summary:
  Classes:  0.00% (0/1)
  Methods:  0.00% (0/2)
  Lines:   94.12% (176/187)

@AlexSkrypnyk
Copy link
Member Author

Code Coverage Report:
  2026-02-17 05:25:08

 Summary:
  Classes:  0.00% (0/1)
  Methods:  0.00% (0/2)
  Lines:   94.12% (176/187)

@AlexSkrypnyk
Copy link
Member Author

Code Coverage Report:
  2026-02-17 05:25:18

 Summary:
  Classes:  0.00% (0/1)
  Methods:  0.00% (0/2)
  Lines:   94.12% (176/187)

@codecov
Copy link

codecov bot commented Feb 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.12%. Comparing base (8b8c24d) to head (2bdb381).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2297      +/-   ##
==========================================
- Coverage   77.69%   77.12%   -0.58%     
==========================================
  Files         117      110       -7     
  Lines        6174     6015     -159     
  Branches       44        0      -44     
==========================================
- Hits         4797     4639     -158     
+ Misses       1377     1376       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@AlexSkrypnyk AlexSkrypnyk merged commit f1d3e92 into main Feb 17, 2026
26 of 28 checks passed
@AlexSkrypnyk AlexSkrypnyk deleted the feature/1559-preso branch February 17, 2026 05:36
@github-project-automation github-project-automation bot moved this from BACKLOG to Release queue in Vortex Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Release queue

Development

Successfully merging this pull request may close these issues.

Convert DS2025 preso to static preso and add it to the maintenance section

1 participant