Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ts rwrite #38

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Ts rwrite #38

wants to merge 14 commits into from

Conversation

cyrilchapon
Copy link

@cyrilchapon cyrilchapon commented Oct 31, 2022

EDIT
Re-pull request from #31

I reverted #29 (see justification there); and merged dependency upgrade conflicts (I had bumped everything to latest major in between)


Project changes

  • Full rewrite to TS
  • Change build from babel to tsc
  • Remove .editorconfig, add prettier, and change eslint-config-google to eslint-config-prettier
    (and configured properly, inspired by Nest.js config)
  • Change pre-commit in favor of more robust husky hooks
  • Refactored source (mostly splits and renames)
  • Removed JSDoc arguments docs in favor of TS ones
    (which fixes options in the process, essentially missing queryInterfaces and queryOptionss)
  • Removed "global" logic for mocha and chai utils,
    they are now simply imported
  • Removed .travisyml in favor of github actions config
  • Added .nvmrc with node 18, configured everything (the build) for Node 18 too
  • Kept the 0 runtime dependency strategy, and added some dev ones accordingly
  • Added sequelize 6.x as a peerDependency (⚠️ Not sure about that one)
  • (Fixed and improved documentation)
  • (Bump major to 2.0.1)

API changes

  • All calls are now curryfied with queryInterface
      replaceEnum(queryInterface)({ /* args */ })
  • sequelizeOptions are now splitted into transactionOptions and queryOptions
    (respectively used for transaction generation, and queries options)
      replaceEnum(queryInterface)({
        // Mainly used for passing enclosing transaction
        // (auto generating a sub-transaction)
        transactionOptions: { transaction: t },
    
        // Used for various query options
        // passed to to all sub-queries
        queryOptions: { logging: true }
    
        // ...other args
      })

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.

1 participant