-
-
Notifications
You must be signed in to change notification settings - Fork 13.2k
Add Copilot instructions #250521
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
Add Copilot instructions #250521
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot wasn't able to review any files in this pull request.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sense! 💯
|
Is it a problem copilot can't review this? |
|
@SMillerDev don't think so no |
5009597 to
04be41e
Compare
04be41e to
94e1de1
Compare
94e1de1 to
8453812
Compare
|
@botantony thanks for the attempt but think this is worth waiting until next stable tag. |
8453812 to
2465afd
Compare
b1a93fa to
de39079
Compare
These will be used for code review.
de39079 to
7571182
Compare
| - name: Cache style cache | ||
| uses: actions/cache@0057852bfaa89a56745cba8c7296529d2fc39830 # v4.3.0 | ||
| with: | ||
| path: /home/linuxbrew/.cache/Homebrew/style | ||
| key: style-cache-${{ matrix.stable && 'stable-' || 'main-' }}${{ github.sha }} | ||
| restore-keys: style-cache-${{ matrix.stable && 'stable-' || 'main-' }} | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have a question: could this step affect the time it takes to complete tap_syntax job? Now it takes ~11 minutes instead of the usual ~4
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops, yes, this was removed for debugging only. Will add it back.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These will be used for code review.