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

*: split functions #53

Merged
merged 1 commit into from
Dec 21, 2023
Merged

*: split functions #53

merged 1 commit into from
Dec 21, 2023

Conversation

thypon
Copy link
Member

@thypon thypon commented Dec 21, 2023

No description provided.

@thypon thypon force-pushed the features/explainBump branch 9 times, most recently from f1e476d to 72c4220 Compare December 21, 2023 12:06
Copy link

New dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
markdown-to-txt 2.0.1 None +3 465 kB ejrbuss

@thypon thypon force-pushed the features/explainBump branch 3 times, most recently from 8e3143e to 78c96b6 Compare December 21, 2023 13:56
@brave brave deleted a comment from github-actions bot Dec 21, 2023
@thypon thypon force-pushed the features/explainBump branch 2 times, most recently from 76b6273 to b7e5bf6 Compare December 21, 2023 20:05
Copy link

[puLL-Merge] - brave/pull-merge@53

Description

This pull request (PR) adds several changes to the codebase. It introduces new functions getConfig, getProperties, getPatch, getRenovatePatch, and filterdiff. It also modifies the runs section in the action.yml file and adds dependencies lodash and markdown-to-txt in the package.json file.

The main motivation for this PR is to enhance the functionality of the puLL-Merge GitHub Action by adding new features and improving existing ones.

Changes

Changes

  • action.yml

    • Added imports for new functions explainPatch, filterdiff, getPatch, getRenovatePatch, getConfig, and getProperties.
    • Replaced the existing getConfig and getProperties function calls with new ones that include more parameters.
    • Removed the getPatch, explainPatch, and related parameters from the function call.
    • Added new logic to fetch the patch body based on whether the PR is from the renovate[bot] or not.
    • Changed the parameter name patch to patchBody in the explainPatch function call.
    • Removed the filterdiffArgs, runIfPrivate, runIfInternal, and related logic.
  • package.json

    • Added dependencies lodash and markdown-to-txt.
  • src/filterdiff.js

    • Added a new file implementing the filterdiff function using spawn to execute the filterdiff command.
  • src/getConfig.js

    • Added a new file implementing the getConfig function that retrieves the content of a file from a GitHub repository.
  • src/getPatch.js

    • Added a new file implementing the getPatch function that fetches the patch body of a pull request from a GitHub repository.
  • src/getProperties.js

    • Added a new file implementing the getProperties function that retrieves the properties of a GitHub repository.
  • src/getRenovatePatch.js

    • Added a new file implementing the getRenovatePatch function that fetches the patch body of a renovate pull request from a GitHub repository.
  • src/openaiExplainPatch.js

    • Removed the filterdiff function from the file.
    • Added imports for the spawn function.
    • Removed the logic for fetching the patch body and replaced it with a parameter patchBody in the explainPatch function.

Security Hotspots

No security hotspots identified in this PR.

@thypon thypon force-pushed the features/explainBump branch from b7e5bf6 to d71c414 Compare December 21, 2023 20:29
@thypon thypon merged commit b613e1d into main Dec 21, 2023
4 of 5 checks passed
@thypon thypon deleted the features/explainBump branch December 21, 2023 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant