Skip to content

Fix/group by with row series number#5054

Merged
fangsmile merged 20 commits intodevelopfrom
fix/groupBy-with-rowSeriesNumber
Mar 18, 2026
Merged

Fix/group by with row series number#5054
fangsmile merged 20 commits intodevelopfrom
fix/groupBy-with-rowSeriesNumber

Conversation

@fangsmile
Copy link
Copy Markdown
Contributor

[中文版模板 / Chinese template]

🤔 This is a ...

  • New feature
  • Bug fix
  • TypeScript definition update
  • Bundle size optimization
  • Performance optimization
  • Enhancement feature
  • Refactoring
  • Update dependency
  • Code style optimization
  • Test Case
  • Branch merge
  • Site / documentation update
  • Demo update
  • Workflow
  • Chore
  • Release
  • Other (about what?)

🔗 Related issue link

💡 Background and solution

📝 Changelog

Language Changelog
🇺🇸 English
🇨🇳 Chinese

☑️ Self-Check before Merge

⚠️ Please check all items below before requesting a reviewing. ⚠️

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • TypeScript definition is updated/provided or not needed
  • Changelog is provided or not needed

🚀 Summary

copilot:summary

🔍 Walkthrough

copilot:walkthrough

const provinces = ['河北', '河南', '山西', '湖北', '内蒙古', '湖南', '广东', '浙江', '四川', '北京', '上海'];

return Array.from(new Array(count)).map((_, i) => {
const salary = Math.floor(5000 + Math.random() * 15000);

Check failure

Code scanning / CodeQL

Insecure randomness High

This uses a cryptographically insecure random number generated at
Math.random()
in a security context.

Copilot Autofix

AI 13 days ago

Copilot could not generate an autofix suggestion

Copilot could not generate an autofix suggestion for this alert. Try pushing a new commit or if the problem persists contact support.

@fangsmile fangsmile merged commit 97541aa into develop Mar 18, 2026
6 of 9 checks passed
@fangsmile fangsmile deleted the fix/groupBy-with-rowSeriesNumber branch March 18, 2026 01:48
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