Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: Documentation for built-in functions in Jenny #2258

Merged
merged 12 commits into from
Jan 8, 2023
Merged

Conversation

st-pasha
Copy link
Contributor

@st-pasha st-pasha commented Jan 8, 2023

Description

This PR adds missing documentation for all built-in functions available in Jenny.

In addition:

  • The bool() function now behaves the same way as the conversion of arguments for an invocation of a user-defined function;
  • Consequently, static sets trueValues/falseValues moved from CommandStorage class into the YarnProject;
  • Added some tests for several other functions.

Checklist

  • I have followed the Contributor Guide when preparing my PR.
  • I have updated/added tests for ALL new/updated/fixed functionality.
  • I have updated/added relevant documentation in docs and added dartdoc comments with ///.
  • [-] I have updated/added relevant examples in examples or docs.

Breaking Change?

  • No, this PR is not a breaking change.

Related Issues

Closes #2236

@st-pasha st-pasha added the jenny Issues/PRs related to the Jenny package label Jan 8, 2023
@st-pasha st-pasha requested a review from spydon January 8, 2023 21:56
@spydon spydon enabled auto-merge (squash) January 8, 2023 22:10
@spydon spydon merged commit 2eac6f5 into main Jan 8, 2023
@spydon spydon deleted the ps.docs-functions branch January 8, 2023 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
jenny Issues/PRs related to the Jenny package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add documentation for builtin function in Jenny
2 participants