Skip to content
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

cli: Skip validations for aggregations #1548

Merged
merged 1 commit into from
Feb 10, 2024
Merged

cli: Skip validations for aggregations #1548

merged 1 commit into from
Feb 10, 2024

Conversation

lutter
Copy link
Collaborator

@lutter lutter commented Dec 15, 2023

This is a placeholder that simply allows @aggregation annotation. The real change should come from using graph-node's validation functionality.

Copy link

changeset-bot bot commented Dec 15, 2023

🦋 Changeset detected

Latest commit: 9ea4027

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@graphprotocol/graph-cli Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

github-actions bot commented Dec 15, 2023

🚀 Snapshot Release (alpha)

The latest changes of this PR are available as alpha on npm (based on the declared changesets):

Package Version Info
@graphprotocol/graph-cli 0.68.0-alpha-20240208230340-9ea4027 npm ↗︎ unpkg ↗︎

Copy link

cloudflare-workers-and-pages bot commented Feb 8, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9ea4027
Status: ✅  Deploy successful!
Preview URL: https://27921ab8.graph-tooling.pages.dev
Branch Preview URL: https://lutter-agg-minimal.graph-tooling.pages.dev

View logs

@lutter
Copy link
Collaborator Author

lutter commented Feb 8, 2024

Fixed linting error and rebased to latest main

@saihaj
Copy link
Member

saihaj commented Feb 8, 2024

Can you please run pnpm changeset create a minor patch for the CLI package so we can generate it for release notes and our automated deploy setup.

also can revert the prettier auto-save change back to true cause I did fix it on main

@lutter lutter force-pushed the lutter/agg-minimal branch from a16e444 to 9053c5d Compare February 8, 2024 23:02
This is a placeholder that simply allows `@aggregation` annotation. The
real change should come from using `graph-node`'s validation functionality.
@lutter lutter force-pushed the lutter/agg-minimal branch from 9053c5d to 9ea4027 Compare February 8, 2024 23:03
@lutter
Copy link
Collaborator Author

lutter commented Feb 8, 2024

Thanks for the review, I just fixed the PR up accordingly

@saihaj saihaj merged commit b3f6a99 into main Feb 10, 2024
11 checks passed
@saihaj saihaj deleted the lutter/agg-minimal branch February 10, 2024 17:14
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.

2 participants