Skip to content

copilot-instructions #5719

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 1 commit into from
Apr 30, 2025
Merged

Conversation

youknowone
Copy link
Member

@youknowone youknowone commented Apr 20, 2025

If you have change requests, do not manually edit the file but ask a language model to add the contents you want to add. That affect machine-readability.

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a new documentation file that outlines guidelines for using GitHub Copilot with the RustPython project. The file covers an overview of the project, repository structure, development notes, coding guidelines, integration examples, and build instructions.

  • Added a new file (.github/workflows/copilot-instructions.md) with detailed instructions.
  • Detailed sections include project overview, testing, integration between Rust and Python, and build commands.
Comments suppressed due to low confidence (1)

.github/workflows/copilot-instructions.md:164

  • Confirm that the target triple 'wasm32-wasip1' is correct; if this is a typo, consider updating it to the intended triple (e.g., 'wasm32-wasi').
cargo build --target wasm32-wasip1 --no-default-features --features freeze-stdlib,stdlib --release

@youknowone youknowone added the skip:ci Skip running the ci label Apr 20, 2025
@youknowone youknowone force-pushed the copilot-instructions branch 2 times, most recently from 2dfba99 to 0447377 Compare April 20, 2025 10:26
@arihant2math
Copy link
Collaborator

According to https://docs.github.com/en/copilot/customizing-copilot/adding-repository-custom-instructions-for-github-copilot

it seems like it's supposed to be .github/copilot-instructions.md not .github/workflows/copilot-instructions.md.

@youknowone youknowone force-pushed the copilot-instructions branch from 0447377 to cb0d31d Compare April 20, 2025 23:50
@youknowone
Copy link
Member Author

huh, Copilot review is so useless not to catch this

@auvipy
Copy link

auvipy commented Apr 23, 2025

huh, Copilot review is so useless not to catch this

it is indeed! is there any way to use claude?

@youknowone
Copy link
Member Author

I never used, so this is not a recommendation.
check products like code rabbit if you are interested in review service.
Doubt for claude at the moment. It is very expensive

@youknowone youknowone merged commit 02f120a into RustPython:main Apr 30, 2025
7 checks passed
@youknowone youknowone deleted the copilot-instructions branch April 30, 2025 05:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip:ci Skip running the ci
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants