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

Update release-notes.md #137

Merged
merged 1 commit into from
Feb 1, 2024
Merged

Update release-notes.md #137

merged 1 commit into from
Feb 1, 2024

Conversation

erosiec
Copy link
Member

@erosiec erosiec commented Feb 1, 2024

Release notes 12.30.23-1.26.24

Release notes 12.30.23-1.26.24
Copy link
Contributor

@gilday gilday left a comment

Choose a reason for hiding this comment

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

lgtm

* New codemod: `use-set-literal` Converts Python set constructions using literal list arguments into more efficient and readable set literals. It simplifies expressions like `set([1, 2, 3])` to `{1, 2, 3}`, enhancing both performance and code clarity. See codemod documentation [here](https://docs.pixee.ai/codemods/python/pixee_python_use-set-literal#pixeepythonuse-set-literal)

#### ☕️ Java
* Added short-circuiting to improve performance of composed codemods
Copy link
Contributor

Choose a reason for hiding this comment

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

What's this change?

Copy link
Member Author

Choose a reason for hiding this comment

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

@gilday It was included in these changelog updates: https://www.notion.so/pixee/Week-Ending-Jan-5-Release-Notes-297aba31619e49eba6992d3732c4a3a1?pvs=4#9304bef4811445d9ace54352b222822a

Do you think this needs more detail to be mentioned here?

Copy link
Contributor

Choose a reason for hiding this comment

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

No, this audience just wants to know that we made something faster. I'm not this audience, so I'll follow up with Arshan to learn more :-)

@erosiec erosiec merged commit 9cd4b3d into main Feb 1, 2024
2 checks passed
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