Skip to content
This repository has been archived by the owner on Mar 12, 2024. It is now read-only.

Test allSettled to allow commenting #432

Merged
merged 1 commit into from
Jan 24, 2024
Merged

Test allSettled to allow commenting #432

merged 1 commit into from
Jan 24, 2024

Conversation

EstebanDalelR
Copy link
Member

Description

This expects all functions to end before returning

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Chore: cleanup/renaming, etc
  • RFC
  • Test

Notes

Possibly the values are returning as empty?

Acceptance

Copy link

vercel bot commented Jan 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
watermelon ✅ Ready (Inspect) Visit Preview Jan 24, 2024 8:39pm

Watermelon AI Summary

This Pull Request addresses a bug where the system is expected to wait for all associated functions to complete before producing an output, which might be related to comment functionality not working as intended when values return prematurely or as empty. It involves a bug fix to ensure proper sequencing and completion of tasks.

GitHub PRs

watermelon is an open repo and Watermelon will serve it for free.
🍉🫶

@@ -437,7 +437,7 @@ export async function POST(request: Request) {
repoName: repo,
});
textToWrite += randomText();
Promise.all([
Promise.allSettled([
// Detect console.logs and its equivalent in other languages
CodeComments
Copy link
Member Author

Choose a reason for hiding this comment

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

Maybe this value is falsy?

Copy link
Collaborator

@baristaGeek baristaGeek left a comment

Choose a reason for hiding this comment

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

LGTM from the theoretical POV.

From the practical one, it's still not working reliably and needs some deeper re-work.

@EstebanDalelR EstebanDalelR merged commit 0f4973a into dev Jan 24, 2024
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants