Skip to content
This repository has been archived by the owner on Jul 23, 2024. It is now read-only.

Subgraph GitHub actions #384

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

iamvigneshwars
Copy link
Collaborator

No description provided.

@iamvigneshwars iamvigneshwars added enhancement New feature or request github_actions Pull requests that update GitHub Actions code labels Mar 11, 2024
@iamvigneshwars iamvigneshwars self-assigned this Mar 11, 2024
.github/workflows/schema.yml Show resolved Hide resolved
backend/compound_library/src/main.rs Outdated Show resolved Hide resolved

- name: check subgraph schema
if: ${{ matrix.service.publish }}
uses: danielsinclair/rover-setup@v1
Copy link
Member

Choose a reason for hiding this comment

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

Can we use a fully qualified version?

Comment on lines +60 to +62
args: subgraph check xchemlab@current
--schema ${{ matrix.service.name }}.graphql
--name ${{ matrix.service.name }}
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
args: subgraph check xchemlab@current
--schema ${{ matrix.service.name }}.graphql
--name ${{ matrix.service.name }}
args: |
subgraph check xchemlab@current
--schema ${{ matrix.service.name }}.graphql
--name ${{ matrix.service.name }}

Comment on lines +70 to +73
args: subgraph publish xchemlab@current
--routing-url http://${{ matrix.service.name }}:80
--schema ${{ matrix.service.name }}.graphql
--name ${{ matrix.service.name }}
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
args: subgraph publish xchemlab@current
--routing-url http://${{ matrix.service.name }}:80
--schema ${{ matrix.service.name }}.graphql
--name ${{ matrix.service.name }}
args: |
subgraph publish xchemlab@current
--routing-url http://${{ matrix.service.name }}:80
--schema ${{ matrix.service.name }}.graphql
--name ${{ matrix.service.name }}

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request github_actions Pull requests that update GitHub Actions code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants