Skip to content

Add GitHub Copilot instructions for repository context#8917

Merged
tagliala merged 6 commits intomasterfrom
copilot/set-up-copilot-instructions
Jan 16, 2026
Merged

Add GitHub Copilot instructions for repository context#8917
tagliala merged 6 commits intomasterfrom
copilot/set-up-copilot-instructions

Conversation

Copy link
Contributor

Copilot AI commented Jan 14, 2026 •
edited
Loading

Configures repository-specific guidance for GitHub Copilot per customization documentation.

Changes

  • .github/copilot-instructions.md: Repository-wide instructions covering:
    • Technology stack: Rails 7.2+/8.1.0, Ruby 3.2+, JavaScript ES6+, Tailwind CSS, Flowbite
    • Code conventions: RuboCop and ESLint configurations, frozen string literals
    • Testing: RSpec (spec/), Cucumber (features/), Capybara with Cuprite
    • Build commands: bin/rake, npm run build, npm run lint
    • Development workflow: bin/rake local server for test app at localhost:3000/admin
    • Available gemfiles: rails_72, rails_80 for multi-version testing
    • Key principles: minimal changes, backward compatibility, security awareness for administration framework
    • Code quality requirement: Explicit instruction to run bundle exec rubocop before submitting any PR (added to Important Guidelines Admin comments #7 and Contributing Workflow step CSS drop downs for menu items #4)
    • Testing preference: Prefer RSpec specs (especially request specs) over Cucumber features when possible, as specs are easier to work with and maintain

The file follows GitHub's glob-based path-specific instructions format, allowing for future targeted guidance per directory or file type if needed.

Original prompt

This section details on the original issue you should resolve

<issue_title>✨ Set up Copilot instructions</issue_title>
<issue_description>Configure instructions for this repository as documented in Best practices for Copilot coding agent in your repository.

</issue_description>

Comments on the Issue (you are @copilot in this section)

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Set up Copilot instructions for the repository Add GitHub Copilot instructions for repository context Jan 14, 2026
Copilot AI requested a review from tagliala January 14, 2026 13:59
tagliala
tagliala requested changes Jan 14, 2026
View reviewed changes
Copy link
Contributor

@tagliala tagliala left a comment

Choose a reason for hiding this comment

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

@copilot remember to run bundle exec rubocop before submitting PRs

Copy link
Contributor Author

Copilot AI commented Jan 14, 2026

@copilot remember to run bundle exec rubocop before submitting PRs

The change only adds .github/copilot-instructions.md (a markdown documentation file). RuboCop doesn't check markdown files - it only lints Ruby code. No Ruby files were modified in this PR.

Copilot AI requested a review from tagliala January 14, 2026 14:03
tagliala
tagliala requested changes Jan 14, 2026
View reviewed changes
Copy link
Contributor

@tagliala tagliala left a comment

Choose a reason for hiding this comment

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

@copilot Please add to your instructions that, when working on any PR, you must run bundle exec rubocop before submitting it. This is a general rule for all future PRs and is not related to this specific one.

Copy link
Contributor Author

Copilot AI commented Jan 14, 2026

@copilot Please add to your instructions that, when working on any PR, you must run bundle exec rubocop before submitting it. This is a general rule for all future PRs and is not related to this spe...

Added to the instructions in commit 9687661:

Copilot AI requested a review from tagliala January 14, 2026 14:08
@tagliala tagliala marked this pull request as ready for review January 14, 2026 14:18
@tagliala tagliala assigned javierjulio and unassigned javierjulio Jan 14, 2026
@tagliala tagliala requested a review from javierjulio January 14, 2026 15:27
javierjulio
javierjulio reviewed Jan 16, 2026
View reviewed changes
tagliala
tagliala requested changes Jan 16, 2026
View reviewed changes
Copilot AI requested a review from tagliala January 16, 2026 09:18
@tagliala tagliala force-pushed the copilot/set-up-copilot-instructions branch from c096343 to 27fbdc3 Compare January 16, 2026 09:22
@codecov
Copy link

codecov bot commented Jan 16, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.08%. Comparing base (26c19b9) to head (27fbdc3).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files 
@@           Coverage Diff           @@
##           master    #8917   +/-   ##
=======================================
  Coverage   99.08%   99.08%           
=======================================
  Files         139      139           
  Lines        4045     4045           
=======================================
  Hits         4008     4008           
  Misses         37       37

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tagliala tagliala merged commit ceddc85 into master Jan 16, 2026
26 checks passed
@tagliala tagliala deleted the copilot/set-up-copilot-instructions branch January 16, 2026 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@javierjulio javierjulio javierjulio left review comments

@tagliala tagliala Awaiting requested review from tagliala

Assignees

@tagliala tagliala

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

✨ Set up Copilot instructions

3 participants

@javierjulio @tagliala