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

Adds higher-level conditions functions #194

Merged
merged 1 commit into from
May 22, 2024
Merged

Conversation

elijahbenizzy
Copy link
Contributor

@elijahbenizzy elijahbenizzy commented May 22, 2024

cond_1 and cond_2
cond_1 or cond_2

Just for fun

Changes

  • adds | and & for conditions

How I tested this

Unit tests

Notes

Checklist

  • PR has an informative and human-readable title (this will be pulled into the release notes)
  • Changes are limited to a single goal (no scope creep)
  • Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future TODOs are captured in comments
  • Project documentation has been updated if adding/changing functionality.

cond_1 and cond_2
cond_1 or cond_2
@elijahbenizzy elijahbenizzy merged commit 7735ada into main May 22, 2024
7 checks passed
@elijahbenizzy elijahbenizzy deleted the merging-conditions branch May 22, 2024 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants