Skip to content

Conversation

@mathiasrw
Copy link
Member

Thank you for the time you are putting into AlaSQL!

@mathiasrw
Copy link
Member Author

@copilot Please make a test file called test/test1740.js and replicate the issue described in #1740

Then see if it has been fixed with the last commit and if not, please work on fixing it.

Copy link
Contributor

Copilot AI commented Nov 30, 2025

@mathiasrw I've opened a new pull request, #2288, to work on those changes. Once the pull request is ready, I'll request review from you.

@mathiasrw
Copy link
Member Author

@copilot

": true,
    "isWebWorker": false
}
/home/runner/_work/alasql/alasql/test


  ........................................................
  ...........................................!

  99 passing (149ms)
  1 failing

  1) Test 046
       FROM as parameter
         FROM array of arrays:

      AssertionError [ERR_ASSERTION]: 15 == 0
      + expected - actual

      -15
      +0
      
      at Context.<anonymous> (test/test046.js:35:11)
      at process.processImmediate (node:internal/timers:483:21)



error Command fai

Copy link
Contributor

Copilot AI commented Nov 30, 2025

@mathiasrw I've opened a new pull request, #2289, to work on those changes. Once the pull request is ready, I'll request review from you.

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.

3 participants