Skip to content
Snippets Groups Projects
  1. Nov 11, 2023
  2. Sep 16, 2023
  3. Sep 13, 2023
  4. Sep 11, 2023
  5. Sep 09, 2023
  6. Sep 05, 2023
  7. Aug 19, 2023
  8. Aug 16, 2023
  9. Aug 13, 2023
  10. Aug 09, 2023
  11. Aug 07, 2023
  12. Jul 18, 2023
  13. Jul 17, 2023
  14. Jun 30, 2023
  15. Jun 28, 2023
  16. Jun 27, 2023
  17. Jun 26, 2023
    • Jed Fox's avatar
      Run ESLint at the top level once (#1202) · ed50e2b3
      Jed Fox authored
      This significantly speeds up `yarn lint` for me. It also ensures we’re
      listing all source files in the project, including the `.eslintrc` files
      and any other files that may be present.
      ed50e2b3
  18. Jun 24, 2023
  19. Jun 09, 2023
  20. May 29, 2023
    • Jed Fox's avatar
      More import-related ESLint rules (#1070) · e660e1e7
      Jed Fox authored
      - Enforce that imports from the same package are merged into a single
      import
      - In `loot-core`, require that imports of other `loot-core` files use a
      relative import (like the vast majority of such imports) rather than
      specifiers starting with `loot-core/` (probably a result of moving files
      out of other packages into `loot-core`)
      e660e1e7
  21. May 28, 2023
  22. May 09, 2023
    • Alberto Gasparin's avatar
      f618055a
    • Jed Fox's avatar
      Enable 'curly' rule (#1015) · 54fa4bcc
      Jed Fox authored
      Multi-line `if`/`for` statements in JS can be confusing since there
      aren’t braces to indicate which code is enclosed in the statement. I set
      the configuration to `multi-line` to enforce usage of braces for
      multi-line statement bodies, but still allow things like `if (foo)
      return;`. I additionally added the `consistent` option to require braces
      for all elements of an if/else chain if one element has it. As you can
      see, this set of options pretty closely matches the existing code style.
      
      I was going to comment in #1008 about this stylistic change but realized
      that it’s (IMO) a little impolite to ask for code style changes unless
      they can be automatically enforced.
      
      Note that `if (foo) { \n return; \n }` is still valid and won’t be
      collapsed. I tried to automatically collapse all such cases but it was a
      lot of files and I didn’t want to pick out the useful from the useless
      differences.
      54fa4bcc
  23. Apr 30, 2023
  24. Apr 21, 2023
  25. Apr 12, 2023
  26. Apr 03, 2023
    • Alberto Gasparin's avatar
      Convert loot-core to TS p1 (#841) · 79ad04dd
      Alberto Gasparin authored
      Part 1 of the conversion. Mostly renaming js to ts and making sure
      things make still sense. Added also handy TS ESLint rules.
      
      In order to support the various .web/.electron/... I ended up adopting
      `index.d.ts` as pattern to share type definition. Let me know if that
      makes sense for you too. Right now the function type definition is
      duplicated, but the solution will be importing from `index.d.ts` and
      using `const fn: FnDef = () => ...` that way we can keep all variants in
      sync from a single type file.
      
      Such rewrite however is better done in another PR otherwise we risk
      confusing git and loosing history (rename + too many changes). Another
      thing that might do in the next PR is convert all files to ESModules, as
      things get confusing between CJS exports, ESM default/named and TS adds
      extra complains.
      79ad04dd
  27. Mar 21, 2023
  28. Mar 05, 2023
  29. Mar 02, 2023
  30. Feb 28, 2023
  31. Jan 25, 2023
  32. Jan 18, 2023
  33. Jan 02, 2023
  34. Dec 30, 2022
  35. Sep 02, 2022
  36. Aug 25, 2022
Loading