Skip to content

Conversation

@regenrek
Copy link
Contributor

@regenrek regenrek commented Nov 4, 2025

Adds a simple bubble sort implementation in JavaScript with example usage.


Note

Add bubble sort implementations in JavaScript and Python with example usage and JS module export.

  • Algorithms:
    • JavaScript (bubble_sort.js):
      • Implement bubbleSort(arr) that returns a sorted copy; includes example CLI usage and module.exports export.
    • Python (bubble_sort.py):
      • Implement bubble_sort(arr) that returns a sorted copy; includes example CLI usage.

Written by Cursor Bugbot for commit 77df777. Configure here.

@vercel
Copy link

vercel bot commented Nov 4, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
imgstructa Ready Ready Preview Nov 4, 2025 11:28am

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

}

// Example usage
if (require.main === module || typeof window === 'undefined') {
Copy link

Choose a reason for hiding this comment

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

Bug: Example Usage Block Causes Environment Checks Failures

The example usage block's condition has a couple of issues: it causes a ReferenceError in browser environments by accessing require when undefined, and in Node.js, the typeof window === 'undefined' check makes the condition always true, causing the example code to run every time the module is imported.

Fix in Cursor Fix in Web

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