Skip to content

copilot: Typos and poor style in README #779

Closed

Description

@ivanperez-keera

Description

The README contains several typos, missing commas, issues with capitalization, and poor wording in some sentences. For example, Copilot is written as Copilot in some places, but copilot in others, even when referring to the project (as opposed to the library). The sentence "It read temperature as a byte", which accompanies a code example, is grammatically incorrect.

To make the README useful and easy to read, we should fix all syntactic and grammatical errors wherever possible.

Type

  • Bug: README contains typos, and other grammar and style errors.

Additional context

None.

Requester

  • Ivan Perez.

Method to check presence of bug

This can only be done by visual inspection, since none of the typos are due to wrong spelling of known English words but, rather, proper names, and some of the issues brought up are due to number not matching between a pronoun and a verb. The following issues are identified:

  • "incorporated in a project or standalone application. The C99 code generated": into a project would sound more natural since incorporated denotes movement or direction.
  • "Catch errors in specifications early using expressive static type system": article missing before "expressive".
  • "Copilot you must install a Haskell compiler (GHC) and the package manager": comma missing after "Copilot".
  • "To use Copilot you must have a Haskell compiler (GHC) and the package manager": comma missing after "Copilot".
  • "There is a TravisCI file at the root of the repository that may help with": the name of the company or service is "Travis CI".
  • "It read temperature as a byte": number does not match between "it" and "read", and "temperature" is missing an article.
  • "facilitates building copilot applications that run on Arduino": "copilot" should be capitalized.
  • "facilitates building copilot applications that run on boards supported by the": "copilot" should be capitalized.
  • "Additionally NASA Langley contracts 80LARC17C0004 and NNL09AA00A supported": comma missing after "Additionally".

Expected result

The README does not contain any grammar or style errors. All the above concerns have been addressed.

Desired result

The README does not contain any grammar or style errors. All the above concerns have been addressed.

Proposed solution

Fix all the typos, grammatical errors and poor style choices in the README.

Further notes

None.

Activity

  1. added
    CR:Type:BugAdmin only: Change request pertaining to error detected
    CR:Status:InitiatedAdmin only: Change request that has been initiated
    on Oct 16, 2026
  2. ivanperez-keera commented on Oct 16, 2026

    @ivanperez-keera
    MemberAuthor

    Change Manager: Confirmed that the issue exists.

  3. added
    CR:Status:ConfirmedAdmin only: Change request that has been acknowledged by the change manager
    and removed
    CR:Status:InitiatedAdmin only: Change request that has been initiated
    on Oct 16, 2026
  4. ivanperez-keera commented on Oct 16, 2026

    @ivanperez-keera
    MemberAuthor

    Technical Lead: Confirmed that the issue should be addressed.

  5. added
    CR:Status:AcceptedAdmin only: Change request accepted by technical lead
    and removed
    CR:Status:ConfirmedAdmin only: Change request that has been acknowledged by the change manager
    on Oct 16, 2026
  6. ivanperez-keera commented on Oct 16, 2026

    @ivanperez-keera
    MemberAuthor

    Technical Lead: Issue scheduled for fixing in Copilot 4.8.2.

    Fix assigned to: @ivanperez-keera.

  7. added
    CR:Status:ScheduledAdmin only: Change requested scheduled
    and removed
    CR:Status:AcceptedAdmin only: Change request accepted by technical lead
    on Oct 16, 2026
  8. added this to the 4.8.2 milestone on Oct 16, 2026
  9. 25 remaining items

  10. ivanperez-keera commented on Oct 16, 2026

    @ivanperez-keera
    MemberAuthor

    Implementor: Solution implemented, review requested.

  11. added
    CR:Status:VerificationAdmin only: Change request that is currently being verified
    and removed
    CR:Status:ImplementationAdmin only: Change request that is currently being implemented
    on Oct 16, 2026
  12. ivanperez-keera commented on Oct 16, 2026

    @ivanperez-keera
    MemberAuthor

    Change Manager: Verified that:

    • Solution is implemented:
      • The code proposed compiles and passes all tests. Details:
        Build log: https://app.travis-ci.com/github/Copilot-Language/copilot/builds/279007866
      • The solution proposed produces the expected result. Details:
        Evaluated by visual inspection of the solution, since there is no convenient mechanism to automatically and deterministically verify that all grammatical errors and style issues have been solved.
    • Implementation is documented. Details:
      Change is to the documentation.
    • Change history is clear.
    • Commit messages are clear.
    • Changelogs are updated.
    • Examples are updated. Details:
      No updates needed; change is to the README.
    • Author is internal or has provided signed CLA.
    • Required version bumps are evaluated. Details:
      Bump not needed; change is to the README and does not affect the public API.
  13. ivanperez-keera commented on Oct 16, 2026

    @ivanperez-keera
    MemberAuthor

    Change Manager: Implementation ready to be merged.

  14. added
    CR:Status:ClosedAdmin only: Change request that has been completed
    and removed
    CR:Status:VerificationAdmin only: Change request that is currently being verified
    on Oct 16, 2026
  15. added a commit that references this issue on Oct 27, 2026
    f0d8d40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

CR:Status:ClosedAdmin only: Change request that has been completedCR:Type:BugAdmin only: Change request pertaining to error detected

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions