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

(example) update hackathon workflow to include node.js setup #5986

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dOrgJelli
Copy link
Contributor

@dOrgJelli dOrgJelli requested a review from a team as a code owner October 27, 2023 11:08
@netlify
Copy link

netlify bot commented Oct 27, 2023

Deploy Preview for auto-gpt-docs ready!

Name Link
🔨 Latest commit d5f0a05
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/653b9dcc927e320008ee6e94
😎 Deploy Preview https://deploy-preview-5986--auto-gpt-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

- name: Read node version from .nvmrc (or use default v18.15)
id: nvm
run: echo NVMRC=$([[ -f .nvmrc ]] && cat .nvmrc || echo "v18.15") >> "$GITHUB_OUTPUT"
working-directory: ${{ matrix.agent-name }}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm assuming here that each agent will be cloned at the root directory, and put under a ./${agent-name} directory

# need to be able to push a new commit to our submission. Let me know please.
GPT_MODEL: "gpt-4-0613"
CONTEXT_WINDOW_TOKENS: 8000
MAX_RESPONSE_TOKENS: 2000
Copy link
Contributor Author

@dOrgJelli dOrgJelli Oct 27, 2023

Choose a reason for hiding this comment

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

Another option to get around this agent-specific CI setup is to have sub-workflows that agents could push to AutoGPT's repo. Could simply invoke them like so:

Custom-Agent-Setup:
  if: ...agent-has-custom-setup...
  uses: ./.github/workflows/agent-setup/${{ matrix.agent-name }}.yaml

I'm not sure if GitHub workflows allows for a conditional uses: statement though...

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Oct 27, 2023
@github-actions
Copy link

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conflicts Automatically applied to PRs with merge conflicts size/m
Projects
Status: 🆕 Needs initial review
Development

Successfully merging this pull request may close these issues.

None yet

1 participant