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

OSS-277-Claude-Inspired-Changes #716

Merged
merged 8 commits into from
Mar 1, 2025
Merged

Conversation

reid-spencer
Copy link
Contributor

Various changes derived from a session with Claude, comprising OSS-277, OSS-278, and OSS-279.

reidspencer added 7 commits February 28, 2025 13:37
* Add ShopifyCart test case
* Teach Finder to find values recursively from the container
* Revise overload checking to be smarter so that only the
  overloads that refer to different kinds of definitions or
  fields that have different type definitions are noted.
* Don't bother warning about missing author references.
* Don't warn about modesl without streaming data.
* Make checking for event and result message generation in
  on clauses better and recursively search the statements
* Make function validation use a specialized check for
  emptiness because its contents are not definitions
@reid-spencer reid-spencer self-assigned this Mar 1, 2025
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


reidspencer seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link

sonarqubecloud bot commented Mar 1, 2025

@reid-spencer reid-spencer merged commit 5796a16 into main Mar 1, 2025
3 checks passed
@reid-spencer reid-spencer deleted the OSS-277-Claude-Inspired-Changes branch March 1, 2025 22:06
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.

2 participants