Skip to content
This repository has been archived by the owner on Jan 9, 2025. It is now read-only.

docs: typo fix Update cairo_precompiles.md #1660

Closed
wants to merge 1 commit into from

Conversation

mdqst
Copy link
Contributor

@mdqst mdqst commented Dec 10, 2024

Description

I noticed a small grammatical issue in the documentation regarding the Cairo precompiles section. The sentence:

"The selector of the Cairo function being called should always be present in the contract called."

was a bit awkward and not quite in line with proper English syntax. It should read:

"The selector of the Cairo function being called should always be present in the called contract."

The change was made to correct the word order and ensure clarity.

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

Fix minor grammatical issue in Cairo precompiles documentation
@Eikix
Copy link
Member

Eikix commented Jan 8, 2025

Sorry for the late reply and lack of review. We've been pausing the development of kakarot for now.

I'm cognizant that you spent some time doing this PR, so thank you. I will have to close it since I am cleaning a bit the repo and preparing for archival.

Archiving the repo will solve issue of people working on stuff while we are not accepting new PRs.
Thanks for understanding

@Eikix Eikix closed this Jan 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants