Skip to content

Remove first-interaction welcome for now - #5384

Merged
IAlibay merged 1 commit into
developfrom
disable-welcome
May 25, 2026
Merged

IAlibay merged 1 commit into
developfrom
disable-welcome

Conversation

@IAlibay

@IAlibay IAlibay commented May 25, 2026 •

Copy link
Copy Markdown
Member

Action seems mostly unmaintained and key issues are not being fixed.

Fixes #

Changes made in this Pull Request:

LLM / AI generated code disclosure

LLMs or other AI-powered tools (beyond simple IDE use cases) were used in this contribution: yes / no

PR Checklist

  • Issue raised/referenced?
  • Tests updated/added?
  • Documentation updated/added?
  • package/CHANGELOG file updated?
  • Is your name in package/AUTHORS? (If it is not, add it!)
  • LLM/AI disclosure was updated.

Developers Certificate of Origin

I certify that I can submit this code contribution as described in the Developer Certificate of Origin, under the MDAnalysis LICENSE.


📚 Documentation preview 📚: https://mdanalysis--5384.org.readthedocs.build/en/5384/

Action seems mostly unmaintained and key issues are not being fixed.
@IAlibay
IAlibay requested review from RMeli and orbeckst May 25, 2026 10:06
@IAlibay

IAlibay commented May 25, 2026

Copy link
Copy Markdown
Member Author

Definitely would be keen to hear @RMeli's thoughts here - I saw CSCS did the same in one of their repos.

@codecov

codecov Bot commented May 25, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.86%. Comparing base (13e8664) to head (dbc498c).
⚠️ Report is 28 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #5384   +/-   ##
========================================
  Coverage    93.85%   93.86%           
========================================
  Files          182      182           
  Lines        22509    22509           
  Branches      3202     3202           
========================================
+ Hits         21125    21127    +2     
+ Misses         922      921    -1     
+ Partials       462      461    -1     

☔ View full report in Codecov by Harness.
📢 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.

@BradyAJohnston BradyAJohnston 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.

Happy to just remove it to stop it clogging up PRs with failed runs for now.

@orbeckst orbeckst 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.

Welcome is nice but debugging it is not, I agree with other efforts being higher priority.

@IAlibay
IAlibay merged commit 523c853 into develop May 25, 2026
25 of 26 checks passed
@IAlibay
IAlibay deleted the disable-welcome branch May 25, 2026 21:04
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.

3 participants