Eric J Ma's Website

How to Craft Stellar Pull Request Summaries with GPT-4

written by Eric J. Ma on 2023-05-13 | tags: gpt4 pull requests code review python llamabot github automation machine learning ai coding programming data science nlp open source software development


Today, I discovered an exciting use case for GPT-4: writing improved pull request messages! In this blog post, I will share the process I followed to achieve this.

First, I utilized llamabot, a Python package that I developed.

Next, I obtained the diff of a pull request. The simplest method to do this is by appending .diff to the end of a PR URL, like so:

https://github.com/pyjanitor-devs/pyjanitor/pull/1256 -> https://github.com/pyjanitor-devs/pyjanitor/pull/1256.diff

Here's the code for generating the summary message:

diff = '''
<diff gets pasted here...>
'''

prompt = f"Given the following pull request diff: {diff},
summarize the pull request changes for me in plain English.
Also, postulate the benefits of these changes.
If the changes are not likely to be beneficial,
state your reasons for why."

from llamabot import SimpleBot
prbot = SimpleBot("You are a bot that answers questions about pull request diffs.")

prbot(prompt)

For PR#1262:

This pull request introduces several changes to the `conditional_join` function in the `janitor/functions/conditional_join.py` file
and updates the corresponding tests in the `tests/functions/test_conditional_join.py` file.
The main changes are as follows:

1. Adds support for "outer" join type in the `how` parameter,
allowing users to perform outer joins in addition to inner, left, and right joins.
2. Deprecates the `sort_by_appearance` parameter,
issuing a warning when it is used.
3. Updates the documentation and comments to reflect the changes made.

The benefits of these changes include:

1. Increased functionality:
Users can now perform outer joins using the `conditional_join` function,
providing more flexibility in their data manipulation tasks.
2. Improved performance:
By deprecating the `sort_by_appearance` parameter,
the function may offer better performance in certain cases,
as it no longer needs to maintain the original order of the data.
3. Clearer documentation:
The updated documentation and comments make it easier for users to understand the function's behavior and the available options.

Overall, these changes are likely to be beneficial,
as they enhance the functionality of the `conditional_join` function
and improve its performance in certain scenarios.

And for PR#1261:

The pull request modifies the `_conditional_join_type_check` function in the `conditional_join.py` file.
The main change is the removal of the `is_categorical_dtype` function from the `permitted_types` set
and the addition of a separate check for categorical data types using `isinstance(left_column.dtype, pd.CategoricalDtype)`.

In plain English, the changes can be summarized as follows:
1. The code now checks if the `left_column.dtype` is a categorical data type separately,
instead of including it in the `permitted_types` set.
2. If the `left_column.dtype` is not a categorical data type,
the code proceeds to check if it is one of the other permitted types (datetime64, numeric, or string).
3. If the `left_column.dtype` is a categorical data type,
the code checks if the categories match up to a permutation between the left and right columns.

The benefits of these changes are:
1. Improved readability and clarity of the code by separating the check for categorical data types from the other permitted types.
2. Ensuring that the correct checks are performed for categorical data types,
which may help prevent potential issues or bugs in the future.

Overall, these changes are likely to be beneficial
as they improve the code's readability and maintainability.

This was an impressive outcome! Previously, I would dread having to (1) write a summary of my PRs, and (2) review PRs with very long diffs. This prompt is an excellent solution to both problems!

How do other devs think about this? As pyjanitor co-maintainer Samuel Oranyeli commented,

I am missing out ...

Summarises my code better ... I could def use this to improve my comments and my code generally ... Now I admit mind blown


Cite this blog post:
@article{
    ericmjl-2023-how-4,
    author = {Eric J. Ma},
    title = {How to Craft Stellar Pull Request Summaries with GPT-4},
    year = {2023},
    month = {05},
    day = {13},
    howpublished = {\url{https://ericmjl.github.io}},
    journal = {Eric J. Ma's Blog},
    url = {https://ericmjl.github.io/blog/2023/5/13/how-to-craft-stellar-pull-request-summaries-with-gpt-4},
}
  

I send out a newsletter with tips and tools for data scientists. Come check it out at Substack.

If you would like to sponsor the coffee that goes into making my posts, please consider GitHub Sponsors!

Finally, I do free 30-minute GenAI strategy calls for teams that are looking to leverage GenAI for maximum impact. Consider booking a call on Calendly if you're interested!