Skip to content

Conversation

@UMMAN2005
Copy link
Contributor

No description provided.

@UMMAN2005 UMMAN2005 requested a review from a team as a code owner May 18, 2025 20:00
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented May 18, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: UMMAN2005 / name: Umman Mammadov (edceff0)

@UMMAN2005
Copy link
Contributor Author

Hi @OpenAssetIO/maintainers, the GitHub Actions workflows are pending approval. Can someone please review and approve them so I can proceed? Thanks!

Copy link
Member

@feltech feltech left a comment

Choose a reason for hiding this comment

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

Thanks for this! Apologies for the delayed review. Text looks good, just a couple of style guideline nits.

@UMMAN2005 UMMAN2005 force-pushed the fix-readme-sample-ref branch from 9de81e7 to 077da5a Compare May 22, 2025 19:13
@UMMAN2005 UMMAN2005 requested a review from feltech May 22, 2025 19:15
@UMMAN2005
Copy link
Contributor Author

Ready to merge :)

Copy link
Member

@feltech feltech left a comment

Choose a reason for hiding this comment

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

Thanks!

@feltech
Copy link
Member

feltech commented May 23, 2025

Ah, one last thing

Merging is blocked

Commits must have verified signatures

You just need to GPG sign your commit.

@UMMAN2005 UMMAN2005 force-pushed the fix-readme-sample-ref branch 3 times, most recently from 8d01bb3 to a509693 Compare May 23, 2025 12:43
Fixes OpenAssetIO#121

Signed-off-by: Umman Mammadov <ummanmemmedov2005@gmail.com>
@UMMAN2005 UMMAN2005 force-pushed the fix-readme-sample-ref branch from a509693 to edceff0 Compare May 23, 2025 12:48
@UMMAN2005 UMMAN2005 requested a review from feltech May 23, 2025 12:50
@UMMAN2005
Copy link
Contributor Author

I added verified signature :)

@feltech feltech merged commit bf0d425 into OpenAssetIO:main May 23, 2025
12 checks passed
@feltech
Copy link
Member

feltech commented May 23, 2025

All merged, thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants