Skip to content
This repository has been archived by the owner on Dec 12, 2024. It is now read-only.

Fix Parameter Naming in Cells Documentation (#49) #50

Closed
wants to merge 1 commit into from
Closed

Fix Parameter Naming in Cells Documentation (#49) #50

wants to merge 1 commit into from

Conversation

TanmayDhobale
Copy link

Overview

This pull request aims to address the inconsistencies in parameter naming within the cells section of the Tact language documentation, as outlined in issue #49.

Changes Made

  • Corrected parameter names in the Builder.storeBool function description to improve clarity.
  • Updated documentation to ensure consistency across the cells section, focusing on parameter naming and descriptions.
  • Ensured all changes align with the existing documentation style for better readability and understanding.

Why This Change Is Necessary

The inconsistency in parameter naming within the documentation could potentially lead to confusion for developers trying to learn or implement the Tact language. By standardizing the naming convention and clarifying function descriptions, we aim to enhance the overall developer experience and facilitate easier learning and usage of the Tact language.

Impact

  • These changes are confined to the documentation and do not affect the underlying functionality of the Tact language or its implementation.
  • Developers referencing the cells documentation will now find a more consistent and clear set of instructions, potentially reducing learning curve and implementation errors.

References

Thank you for considering this pull request. I'm open to feedback and willing to make further adjustments to ensure the documentation improvements align well with the project's standards and expectations.

@TanmayDhobale TanmayDhobale closed this by deleting the head repository Feb 28, 2024
@TanmayDhobale
Copy link
Author

can someone check this

@novusnota
Copy link
Member

@TanmayDhobale hey, not anymore, since you deleted the head repo. Besides, the original issue #49 was fixed earlier via a different PR. Thank you for your time though, there are many other issues you may help with in the future! But please wait a bit before #79 gets resolved, as it would involve many files and folders moving around :)

@TanmayDhobale
Copy link
Author

Ahh got it

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