Skip to content

Omics updates #9560

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 2, 2025
Merged

Omics updates #9560

merged 1 commit into from
Jul 2, 2025

Conversation

devlinbd2
Copy link
Contributor

Correct the broken links in the Omics CLI examples

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Correct the broken links in the Omics CLI examples
@kdaily kdaily added pr:work-in-progress This PR is a draft and needs further work. documentation This is a problem with documentation. labels Jun 30, 2025
@codecov-commenter
Copy link

codecov-commenter commented Jun 30, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.40%. Comparing base (68039fa) to head (e36eb0d).
Report is 80 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #9560   +/-   ##
========================================
  Coverage    93.40%   93.40%           
========================================
  Files          211      211           
  Lines        17012    17020    +8     
========================================
+ Hits         15890    15898    +8     
  Misses        1122     1122           

☔ 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.

@elysahall elysahall self-requested a review July 1, 2025 16:16
Copy link
Collaborator

@elysahall elysahall left a comment

Choose a reason for hiding this comment

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

Thank you for contributing to the AWS CLI examples.

These examples LGTM, sending for maintainer review.

@elysahall elysahall assigned kdaily and unassigned elysahall Jul 1, 2025
@elysahall elysahall added pr:needs-review This PR needs a review from a Member. and removed pr:work-in-progress This PR is a draft and needs further work. labels Jul 1, 2025
@kdaily
Copy link
Member

kdaily commented Jul 1, 2025

LGTM! Will port to v2 and merge both once that completes.

https://github.com/aws/aws-cli/actions/runs/16012011290

@github-actions github-actions bot mentioned this pull request Jul 1, 2025
2 tasks
@kdaily kdaily added pr:ready-to-merge This PR is ready to be merged. and removed pr:needs-review This PR needs a review from a Member. labels Jul 1, 2025
@kdaily kdaily merged commit b13c9d5 into aws:develop Jul 2, 2025
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation This is a problem with documentation. pr:ready-to-merge This PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants