Skip to content
Snippets Groups Projects

Fixing linting errors

Merged Fixing linting errors
franklygeorge/fc-pregmod:fixing-linting-errors into pregmod-master
All threads resolved!
Merged Frankly George requested to merge franklygeorge/fc-pregmod:fixing-linting-errors into pregmod-master
All threads resolved!

Fixed some warnings and error that ESLint was flagging.

Also @null your code in src/npc/generateSlaveBot.js contained a lot of linting errors. I fixed most of them but there were a few that I wasn't sure what you were trying to do. These have been marked with // FIXME: @null .... Also if your editor supports an ESLint plugin you should really look into getting it set up.

Merge request reports

Approval is optional

Merged by PregmodderPregmodder 1 year ago (Feb 7, 2024 8:06pm UTC)

Merge details

  • Changes merged into pregmod-master with 534dde92 (commits were squashed).
  • Deleted the source branch.

Pipeline #75974 passed

Stage: build

Pipeline passed for 534dde92 on pregmod-master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Frankly George added 1 commit

    added 1 commit

    • d6ae818b - SimpleVCheck() now returns a string instead of undefined

    Compare with previous version

  • Frankly George resolved all threads

    resolved all threads

  • Frankly George added 10 commits

    added 10 commits

    • d6ae818b...8e9a3b81 - 9 commits from branch pregmodfan:pregmod-master
    • 0c81b82f - Merge branch fc-pregmod:pregmod-master into fixing-linting-errors

    Compare with previous version

  • Pregmodder merged

    merged

  • Pregmodder mentioned in commit 534dde92

    mentioned in commit 534dde92

    • Please register or sign in to reply