Skip to content

docs: Add note about organizational token policies #5161

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

Merged
merged 2 commits into from
Nov 21, 2024

Conversation

neubig
Copy link
Contributor

@neubig neubig commented Nov 21, 2024

Added a note in the resolver README about the need to configure organizational token policies when working with organizational repositories. This helps users understand that they may need to set up additional permissions at the organization level before personal access tokens will work.


To run this PR locally, use the following command:

docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:fd3a87e-nikolaik   --name openhands-app-fd3a87e   docker.all-hands.dev/all-hands-ai/openhands:fd3a87e

@neubig neubig added the lint-fix Attempts to fix lint issues on the PR label Nov 21, 2024
@neubig
Copy link
Contributor Author

neubig commented Nov 21, 2024

Linting is failing, it needs to be fixed.

@neubig neubig added fix-me Attempt to fix this issue with OpenHands and removed lint-fix Attempts to fix lint issues on the PR labels Nov 21, 2024
Copy link
Contributor

OpenHands started fixing the pr! You can monitor the progress here.

@openhands-agent
Copy link
Contributor

New OpenHands update

Copy link
Contributor

The workflow to fix this issue encountered an error. Please check the workflow logs for more information.

@neubig
Copy link
Contributor Author

neubig commented Nov 21, 2024

Now tests are failing. Please fix tests (and make sure that linting still passes).

@neubig neubig added fix-me Attempt to fix this issue with OpenHands and removed fix-me Attempt to fix this issue with OpenHands labels Nov 21, 2024
Copy link
Contributor

OpenHands started fixing the pr! You can monitor the progress here.

Copy link
Contributor

The workflow to fix this issue encountered an error. Openhands failed to create any code changes.

@neubig neubig force-pushed the add-org-token-policy-note branch from 20be6e2 to fd3a87e Compare November 21, 2024 03:36
@neubig neubig enabled auto-merge (squash) November 21, 2024 03:38
@neubig neubig merged commit 12ed523 into main Nov 21, 2024
12 checks passed
@neubig neubig deleted the add-org-token-policy-note branch November 21, 2024 04:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix-me Attempt to fix this issue with OpenHands
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants