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

Fool me issue fix #578

Merged
merged 21 commits into from
Aug 29, 2024
Merged

Fool me issue fix #578

merged 21 commits into from
Aug 29, 2024

Conversation

mgtm98
Copy link
Collaborator

@mgtm98 mgtm98 commented Aug 17, 2024

Description

@mgtm98
Copy link
Collaborator Author

mgtm98 commented Aug 26, 2024

Hi @marsninja, This PR contains

  • fool_me issue fix
  • fool_me2 issue fix (class that have the fool me issue)
  • List & Dict expression symbol linking (IndexSlice type)

marsninja
marsninja approved these changes Aug 29, 2024
View reviewed changes
Copy link
Contributor

@marsninja marsninja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beautiful!

@marsninja marsninja merged commit 2ef73cf into main Aug 29, 2024
4 checks passed
@marsninja marsninja deleted the fool_me_issue branch August 29, 2024 22:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@marsninja marsninja marsninja approved these changes

@kugesan1105 kugesan1105 Awaiting requested review from kugesan1105

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mgtm98 @marsninja @kugesan1105