Skip to content

Add copilot instructions#3390

Merged
ildyria merged 2 commits intomasterfrom
copilot-instructions
Jun 1, 2025
Merged

Add copilot instructions#3390
ildyria merged 2 commits intomasterfrom
copilot-instructions

Conversation

@ildyria
Copy link
Copy Markdown
Member

@ildyria ildyria commented May 31, 2025

This pull request updates the coding guidelines in the .github/copilot-instructions.md file to clarify standards for PHP files and variable usage in requests.

Updates to coding guidelines:

  • Added a requirement for all new PHP files to include a license header and a single blank line after the opening PHP tag.
  • Specified that variable names should follow the snake_case naming convention.
  • Clarified that in requests, if a user is provided by the query, it should be assigned to $this->user2, while $this->user is reserved for the user making the request.

@ildyria ildyria requested a review from a team as a code owner May 31, 2025 10:47
@ildyria ildyria added the Review: easy Easy review expected: probably just need a quick to go through. label May 31, 2025
Copy link
Copy Markdown

@RonnieTaz RonnieTaz left a comment

Choose a reason for hiding this comment

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

Prompt looks good to me

@ildyria ildyria merged commit d38a849 into master Jun 1, 2025
3 checks passed
@ildyria ildyria deleted the copilot-instructions branch June 1, 2025 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Review: easy Easy review expected: probably just need a quick to go through.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants