Skip to content

rename ci jobs for clarity#9

Merged
Yaswant Pradhan (yaswant) merged 3 commits intoMetOffice:mainfrom
james-bruten-mo:rename_ci
Oct 2, 2025
Merged

rename ci jobs for clarity#9
Yaswant Pradhan (yaswant) merged 3 commits intoMetOffice:mainfrom
james-bruten-mo:rename_ci

Conversation

@james-bruten-mo
Copy link
Copy Markdown
Collaborator

PR Summary

Code Reviewer:

Rename the CI steps for clarity

Code Quality Checklist

(Some checks are automatically carried out via the CI pipeline)

  • I have performed a self-review of my own code
  • My code follows the project's style guidelines
  • Comments have been included that aid undertanding and enhance the
    readability of the code
  • My changes generate no new warnings

Testing

  • I have tested this change locally, using the rose-stem suite
  • If any tests fail (rose-stem or CI) the reason is understood and
    acceptable (eg. kgo changes)
  • I have added tests to cover new functionality as appropriate (eg. system
    tests, unit tests, etc.)

trac.log

Security Considerations

  • This change does not introduce security vulnerabilities
  • I have reviewed the code for potential security issues
  • Sensitive data is properly handled (if applicable)
  • Authentication and authorisation are properly implemented (if applicable)

Performance Impact

  • Performance of the code has been considered and, if applicable, suitable
    performance measurements have been conducted

Contributor License Agreement (CLA)

AI Assistance and Attribution

  • Some of the content of this change has been produced with the assistance
    of Generative AI tool name (e.g., Met Office Github Copilot Enterprise,
    Github Copilot Personal, ChatGPT GPT-4, etc) and I have followed the
    Simulation Systems AI policy (including
    attribution labels)

Documentation

  • Where appropriate I have updated documentation related to this change and
    confirmed that it builds correctly

Code Review

  • All dependencies have been resolved
  • Related Issues are properly linked and addressed
  • CLA compliance is confirmed
  • Code quality standards are met
  • Tests are adequate and passing
  • Documentation is complete and accurate
  • Security considerations have been addressed
  • Performance impact is acceptable

@james-bruten-mo James Bruten (james-bruten-mo) added the crSet This is added to a pull request once the SSD team have assigned a code reviewer. label Oct 2, 2025
@yaswant Yaswant Pradhan (yaswant) merged commit bd54671 into MetOffice:main Oct 2, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

crSet This is added to a pull request once the SSD team have assigned a code reviewer.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants