Skip to content

Added GitHub Copilot Instructions#155

Merged
mpivchev merged 2 commits intodevelopfrom
154-copilot-instructions
Aug 21, 2025
Merged

Added GitHub Copilot Instructions#155
mpivchev merged 2 commits intodevelopfrom
154-copilot-instructions

Conversation

@i2h3
Copy link
Collaborator

@i2h3 i2h3 commented Jul 15, 2025

No description provided.

Signed-off-by: Iva Horn <iva.horn@icloud.com>
@i2h3 i2h3 requested a review from Copilot July 15, 2025 12:46
@i2h3 i2h3 self-assigned this Jul 15, 2025
@i2h3 i2h3 added the enhancement New feature or request label Jul 15, 2025

This comment was marked as outdated.

@i2h3 i2h3 force-pushed the 154-copilot-instructions branch from e146768 to 5c7ee3e Compare July 15, 2025 12:50
@i2h3 i2h3 requested a review from Copilot July 15, 2025 12:51

This comment was marked as outdated.

@i2h3 i2h3 requested review from marinofaggiana and mpivchev July 15, 2025 12:56
@i2h3 i2h3 force-pushed the 154-copilot-instructions branch from 5c7ee3e to c92f179 Compare July 15, 2025 12:56
@i2h3 i2h3 requested a review from Copilot July 15, 2025 12:57

This comment was marked as outdated.

@i2h3 i2h3 force-pushed the 154-copilot-instructions branch from c92f179 to b089a53 Compare July 15, 2025 12:59
@i2h3 i2h3 requested a review from Copilot July 15, 2025 12:59

This comment was marked as outdated.

Signed-off-by: Iva Horn <iva.horn@icloud.com>
@i2h3 i2h3 force-pushed the 154-copilot-instructions branch from b089a53 to 45d5bd0 Compare July 15, 2025 13:04
@i2h3 i2h3 requested a review from Copilot July 15, 2025 13:04
Copy link

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 set of contributor guidelines for the Swift-based iOCNotes project and removes an old localization configuration file.

  • Adds .github/copilot-instructions.md with project overview, code standards, development flow, repository structure, and key guidelines for contributors.
  • Removes .bartycrouch.toml localization config—ensure the localization pipeline is updated accordingly.

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
.github/copilot-instructions.md New contributor guide outlining project purpose, SwiftLint enforcement, build/test commands, and repo layout
.bartycrouch.toml Deleted obsolete BartyCrouch configuration for localization tasks
Comments suppressed due to low confidence (3)

.github/copilot-instructions.md:9

  • The note says SwiftLint runs only on iOCNotes/, but the swiftlint command lints the entire project by default—either scope the command (e.g., swiftlint autocorrect --path iOCNotes/) or update the text to reflect the actual behavior.
- Run `swiftlint` before committing any changes to ensure proper code formatting

.github/copilot-instructions.md:30

  • [nitpick] Consider ensuring this numeric list is rendered as a proper Markdown ordered list by adding a blank line before it—this improves readability in rendered views.
1. Follow Swift best practices and idiomatic patterns

Copy link
Collaborator

@mpivchev mpivchev left a comment

Choose a reason for hiding this comment

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

Nice!

@mpivchev mpivchev merged commit b211462 into develop Aug 21, 2025
1 check passed
@i2h3 i2h3 deleted the 154-copilot-instructions branch August 21, 2025 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments