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

docs: fix broken links in README and add CONTRIBUTING.md #5

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

devin-ai-integration[bot]
Copy link

Fix Broken Links in README and Add CONTRIBUTING Guide

This PR fixes several broken links in the README.md file and adds a CONTRIBUTING.md guide.

Changes

  • Updated package links to use fss- prefix instead of agent- prefix to match actual directory names
  • Created CONTRIBUTING.md with standard open source contribution guidelines
  • Fixed PKPToolPolicyRegistry.sol path reference

Testing

  • Verified all updated package links match existing directory structure
  • Confirmed CONTRIBUTING.md follows standard open source practices
  • Checked that external link to Lit faucet is working

Link to Devin run: https://app.devin.ai/sessions/ccde4bb9d3d74852a938c9b4d37ee324

- Update package links to use fss- prefix instead of agent-
- Create CONTRIBUTING.md with standard guidelines
- Fix PKPToolPolicyRegistry.sol path

Co-Authored-By: anson@litprotocol.com <anson@litprotocol.com>
Copy link
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Copy link
Author

The CI failure appears to be a configuration issue with nx affected commands. The command is trying to compare against 'main' branch, but our PR is targeting 'v3'.

I suggest updating the CI workflow to use the correct base branch. This can be done by setting NX_BASE=origin/v3 in the workflow environment.

For now, since this PR only contains documentation changes (fixing broken links in README.md and adding CONTRIBUTING.md), these changes don't affect any package code and shouldn't require nx affected checks. Could we get this PR reviewed and merged, and then address the CI configuration in a separate PR?

@glitch003 glitch003 changed the base branch from v3 to main December 28, 2024 21:15
Copy link

nx-cloud bot commented Dec 28, 2024

View your CI Pipeline Execution ↗ for commit 60cda86.

Command Status Duration Result
nx affected -t lint test build ❌ Failed 15s View ↗

☁️ Nx Cloud last updated this comment at 2024-12-28 21:25:04 UTC

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.

0 participants