Skip to content

compiler: more cleanup#41

Merged
georgebisbas merged 6 commits intomasterfrom
compiler_more
Dec 5, 2023
Merged

compiler: more cleanup#41
georgebisbas merged 6 commits intomasterfrom
compiler_more

Conversation

@georgebisbas
Copy link
Copy Markdown
Collaborator

No description provided.

@georgebisbas georgebisbas added the tests related to adding/editing/deleting tests label Nov 28, 2023
Copy link
Copy Markdown
Collaborator

@PapyChacal PapyChacal left a comment

Choose a reason for hiding this comment

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

Looks good to me !
Not familiar with the tricky bits, but I guess we have CI to trust now 🙂

@georgebisbas georgebisbas changed the title compiler: more rewrites compiler: more cleanup Nov 28, 2023
Copy link
Copy Markdown
Collaborator Author

@georgebisbas georgebisbas left a comment

Choose a reason for hiding this comment

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

@PapyChacal I added some more commits; please have another look.

Copy link
Copy Markdown
Collaborator

@PapyChacal PapyChacal left a comment

Choose a reason for hiding this comment

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

Idem!
Really nice to see this cleaned up 😊

Copy link
Copy Markdown
Collaborator

@PapyChacal PapyChacal left a comment

Choose a reason for hiding this comment

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

Another approval after last commits!

Copy link
Copy Markdown

@AntonLydike AntonLydike left a comment

Choose a reason for hiding this comment

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

👍

@georgebisbas georgebisbas merged commit 107aefd into master Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests related to adding/editing/deleting tests

Projects

Development

Successfully merging this pull request may close these issues.

3 participants