Skip to content

Conversation

@pinkie-ai
Copy link
Contributor

Description

Adding Meridian to the ecosystem page.

Tests

N/A

Checklist

  • [Y] I have formatted and linted my code
  • [Y] All new and existing tests pass
  • [Y] My commits are signed (required for merge) -- you may need to rebase if you initially pushed unsigned commits

@cb-heimdall
Copy link

cb-heimdall commented Sep 23, 2025

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@vercel
Copy link

vercel bot commented Sep 23, 2025

@pinkie-ai is attempting to deploy a commit to the Coinbase Team on Vercel.

A member of the Team first needs to authorize it.

apmcdermott
apmcdermott previously approved these changes Sep 30, 2025
@cb-heimdall
Copy link

Review Error for ajmeese7 @ 2025-09-30 13:14:32 UTC
User must have write permissions to review

@apmcdermott
Copy link
Contributor

@pinkie-ai LGTM but you need to sign all your commits.

pinkie-ai and others added 6 commits October 1, 2025 06:49
* fix the x402 bottom image with a white font for the text
* bumping version to 0.6.5 for release

* fix: Update the x402 bottom image from black front -> white front (#410)

* fix the x402 bottom image with a white font for the text
The GitHub Actions workflow `.github/workflows/check_python.yml` had
a nested `steps:` key inside the `test-python` job’s `steps` list,
causing a YAML parse error:

  (Line: 11, Col: 7): A mapping was not expected

This change removes the redundant `steps:` and ensures all steps are properly listed with `- name:` or `- uses:` entries, restoring workflow syntax validity so the CI pipeline can run.
The published x402 package does not contain an index.d.ts file leading to Typescript errors.

e.g.:
```
Could not find a declaration file for module 'x402'. '/Users/dylanfiedler/Documents/ario/ar-io/ar-io-node/node_modules/x402/dist/esm/index.js' implicitly has an 'any' type.
  Try `npm i --save-dev @types/x402` if it exists or add a new declaration (.d.ts) file containing `declare module 'x402';`ts(7016)
```

This PR updates it to use cjs types by default.

fix(package.json): use cjs by default in package.json
@pinkie-ai
Copy link
Contributor Author

@apmcdermott Signed, thanks for the flag!

@apmcdermott
Copy link
Contributor

@pinkie-ai Ahh sorry to be That Guy but it looks like you haven't uploaded your public signing key to gh.
https://docs.github.com/en/authentication/managing-commit-signature-verification/telling-git-about-your-signing-key

@pinkie-ai
Copy link
Contributor Author

Apologies, uploaded now :)

@apmcdermott
Copy link
Contributor

@pinkie-ai Need you to resolve those conflicts but otherwise you're all set

@apmcdermott apmcdermott self-assigned this Oct 1, 2025
@pinkie-ai
Copy link
Contributor Author

pinkie-ai commented Oct 2, 2025

@apmcdermott Lol sorry for the back and forth here, resolved and merged

This reverts commit 236d01c, reversing
changes made to 1eb9a5e.
@apmcdermott apmcdermott merged commit 053e0eb into coinbase:main Oct 3, 2025
11 of 13 checks passed
apmcdermott added a commit that referenced this pull request Oct 3, 2025
apmcdermott added a commit that referenced this pull request Oct 3, 2025
@apmcdermott
Copy link
Contributor

Hey @pinkie-ai you'll need to create a new PR with your changes. That last merge commit seems to have deleted a bunch of code.

@pinkie-ai
Copy link
Contributor Author

Re-did everything on this PR: #437

Huge appologies for the mess this PR has been. I appreciate your continued support @apmcdermott

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

8 participants