This repository has been archived by the owner on Mar 12, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 14
v2.1.2 #355
Merged
Merged
v2.1.2 #355
Changes from all commits
Commits
Show all changes
120 commits
Select commit
Hold shift + click to select a range
fc8b365
fix: upgrade airtable from 0.11.6 to 0.12.1
snyk-bot b9029f6
Fix/v2.0.0 merge conflicts (#278)
EstebanDalelR c0a6103
Merge branch 'dev' into snyk-upgrade-2fb799419e235800c37aacf6ded49ad0
EstebanDalelR 9a29a68
Build a prompt that produces the desired results
baristaGeek 86c81f7
Flag PRs
baristaGeek 4d0b5bc
Change comparison string for detecting already labeled PRs
baristaGeek 1d246b0
Fix/double cloud id getting (#271)
EstebanDalelR 835e833
Change hardcoded pr title and number to the ones stored on GitHubPRs[0
baristaGeek aa3ce80
Remove console logs
baristaGeek 6bbba41
Chore/delete unused files (#283)
EstebanDalelR b174fd6
Feature/new integrations script (#272)
EstebanDalelR 30dabb9
Feature/asana integration (#273)
EstebanDalelR 25e1350
Move logic to actions/github
baristaGeek 7aa4cad
Fix parameter names
baristaGeek 415b917
Merge pull request #275 from watermelontools/snyk-upgrade-2fb799419e2…
baristaGeek 7095af1
Merge branch 'dev' into feature/Watermelon-as-reviewer-0
baristaGeek 63bcbbb
Feature/improve logging (#291)
EstebanDalelR 8ccb1c7
Feature/streamline new integrations (#292)
EstebanDalelR 5c5c103
Remove running in the extension
baristaGeek ee21e0d
Run sync
baristaGeek 033e3eb
Change threshold back to 9
baristaGeek d05c0f3
Update route.ts (#296)
EstebanDalelR 9aaa301
Update route.ts (#295)
EstebanDalelR 62ab841
Update responses.ts (#294)
EstebanDalelR 1b2c123
Update redirect.tsx (#293)
EstebanDalelR 82a2093
Feature/intellij login (#286)
EstebanDalelR fcb992d
Refactor Watermelon as reviewer 0 to work sync and prevent race condi…
baristaGeek 0e13788
Remove user prompt
baristaGeek 058f1a2
Couple logic inside a single function
baristaGeek 047d4d1
Merge branch 'dev' into feature/Watermelon-as-reviewer-0
baristaGeek aa99e79
Feature/intellij data mapping (#298)
EstebanDalelR f433f20
Make function async again, use installation octokit, and add auto lab…
baristaGeek 2211ba9
Track pr rating per repo on PostHog
baristaGeek 229e3d0
Add param validation
baristaGeek 9693c5d
Merge branch 'dev' into feature/Watermelon-as-reviewer-0
EstebanDalelR 9d37d0f
Merge pull request #282 from watermelontools/feature/Watermelon-as-re…
baristaGeek ef4d61b
Feature/encapsulated tracking (#297)
EstebanDalelR 4569bd6
Chore/2.0.1 merge conflicts (#299)
EstebanDalelR f0458e2
Merge branch 'main' into dev
EstebanDalelR c4e9644
v2.0.1 (#281) (#300)
EstebanDalelR eccc506
Add date to textToWrite for the GitHub service
baristaGeek 7647a25
Calculate relative date
baristaGeek 7fe4841
Print PR author
baristaGeek 1ef114e
Remove conole logs
baristaGeek 0e09202
Remove organization as a mandtory paramater for running the GH app (#…
baristaGeek d6f99d6
REmove github special case
EstebanDalelR 51dfe6f
Fix helper text
EstebanDalelR eb45693
REmove github special case (#303)
EstebanDalelR 0f4e18f
Merge branch 'dev' into fix/broken-deactivation-text
EstebanDalelR 7fbb7af
Merge pull request #304 from watermelontools/fix/broken-deactivation-…
baristaGeek 1ac634b
Merge branch 'dev' into feature/add-ownership-and-date
baristaGeek 3f91367
Merge pull request #301 from watermelontools/feature/add-ownership-an…
baristaGeek efc1be6
Add conditional if the count passes 500
baristaGeek e0d2894
Feature/unauthorized tracking (#309)
EstebanDalelR 8b376fe
Feature/random suggestion text on markdown (#310)
EstebanDalelR 5404ad4
Replace all indexed information with invitation to pay
baristaGeek 7e1ad3d
Remove the logic from the markdown helper
baristaGeek 04f185a
Feature/intellijdownload (#312)
EstebanDalelR 9fdaa47
Remove console log
baristaGeek e20efd1
Unwrapp comenting logic from being below the threshold. Replace comme…
baristaGeek 44bf4ef
Create and call the new stored procedure (increment_owner_github_app_…
baristaGeek ef7dc57
Merge pull request #311 from watermelontools/bugfix/bugfix-not-showin…
baristaGeek 84c0dd8
Merge branch 'dev' into bugfix/user-or-org-to-gh-app
baristaGeek a6b47ba
Merge pull request #313 from watermelontools/bugfix/user-or-org-to-gh…
baristaGeek 3474cb0
Allow empty responses (#317)
EstebanDalelR c0932ed
Make prompt multi-line to make its iteration easier
baristaGeek cd57b53
Bias prompt towards conciseness
baristaGeek 70b92ef
Remove role user prompt for simplicity
baristaGeek c78988e
Merge into a single prompt, and bias the prompt even more towards con…
baristaGeek 3b5a068
Remove github param validation because of the recent permission changes
baristaGeek 52784fe
Remove prod console logs
baristaGeek 0a18c09
Add logic that removes outdated PR labels
baristaGeek 3aff27c
Change the take a deeper dive label emoji
baristaGeek cca1d2c
Merge pull request #322 from watermelontools/feature/trinary-pr-labeling
baristaGeek dab95a9
console.log() file patches
baristaGeek e336e75
Detect console logs and its equivalent in other programming languages…
baristaGeek fd187fe
Comment the PR if there's a console log detected
baristaGeek ec591fa
Remove console logs
baristaGeek 161bc91
remove console log
baristaGeek 9d68579
update prompt
baristaGeek 0a3a481
update prompt
baristaGeek 63b0933
Extract functions and texts (#323) (#325)
EstebanDalelR b937bf3
Ommit console logs that are in code comments
baristaGeek 81380cd
remove old logic
baristaGeek c01fb42
Restric rating to a maximum of 8 if console log comments are found
baristaGeek c86a96a
Merge the commits into the prompt list
baristaGeek 48970fb
Update utils/actions/detectConsoleLogs.ts
baristaGeek d69514b
Promise.allSettled octokit call
baristaGeek ab41609
Merge branch 'dev' into feature/search-console-logs
baristaGeek 3eb21e2
Concat with different syntax
baristaGeek 955413a
fix merge conflict
baristaGeek d642ac4
Merge pull request #324 from watermelontools/feature/search-console-logs
baristaGeek afb3999
Chore/extract PR grading (#333)
EstebanDalelR df0bb3e
bring the user prompt back
baristaGeek 28a53a4
Add consistent CSS for boxes regardless of login state + prettify design
baristaGeek b361795
Style the boxes for all integrations
baristaGeek cfe74c9
Merge pull request #321 from watermelontools/enhancement/improve-prom…
baristaGeek 2fd51f0
Make IDE boxes consistenet with the integraitons boxes
baristaGeek b86be53
Rebuild sidebar
baristaGeek 8e33d77
Redesign the header
baristaGeek cf21362
Make final adjustments to header
baristaGeek 894cc1a
Make instruction more specific and add the C++ exmaple
baristaGeek c62f679
Merge pull request #340 from watermelontools/chore/detect-cpp-console…
baristaGeek 36bfcfd
Create getTeamGitHub.ts (#338)
EstebanDalelR eae03a0
Create createTeamAndMatchUser.ts (#337)
EstebanDalelR eaa2ec6
Conditionally render placeholder (#342)
EstebanDalelR 7925771
Feature/comment specific line diffs (#331)
baristaGeek 72b539f
Watermelon using app token (team tool) (#326)
EstebanDalelR fd21a53
Re-style team page
baristaGeek ec94f5d
Style settings screen
baristaGeek ace2080
Merge branch 'dev' into enhancement/beautify-dashboard
baristaGeek 4ad49a4
Pull changes
baristaGeek 9c07826
Merge pull request #339 from watermelontools/enhancement/beautify-das…
baristaGeek 78eac4c
Merge branch 'main' into dev
baristaGeek 8d8c315
Fix indexing issue for the single line file border-case (#346)
baristaGeek 4739d5b
Delete app/(loggedIn)/settings/layout.tsx (#348)
EstebanDalelR adcc4a9
Add tracking to keys existing
EstebanDalelR 5015186
Fix/broken login getting (#353)
EstebanDalelR 3d25273
Chore/readme update messaging and strategy (#352)
baristaGeek 754aeea
Merge branch 'main' into dev
EstebanDalelR File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -163,6 +163,7 @@ export default async function detectConsoleLogs({ | |
filePatch: file.patch ?? "", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This file contains at least one console log. Please remove any present. |
||
individualLine | ||
}) | ||
|
||
return octokit.request( | ||
"POST /repos/{owner}/{repo}/pulls/{pull_number}/reviews", | ||
{ | ||
|
@@ -176,7 +177,6 @@ export default async function detectConsoleLogs({ | |
{ | ||
path: file.filename, | ||
position: consoleLogPosition || 1, // comment at the beggining of the file by default | ||
|
||
body: "This file contains at least one console log. Please remove any present.", | ||
}, | ||
], | ||
|
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file contains at least one console log. Please remove any present.