Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

Allow users to enable @truffle/db as a system of record for their Truffle projects #3784

Merged
merged 113 commits into from
Feb 26, 2021

Conversation

gnidan
Copy link
Contributor

@gnidan gnidan commented Feb 9, 2021

With this PR, adding db: { enabled: true } will turn on @truffle/db for use with truffle compile, truffle migrate, etc.

This only adds data to @truffle/db right now; nothing inside Truffle is currently configured to read from @truffle/db.

@gnidan gnidan changed the title Allow users to use @truffle/db as system of record inside Truffle Allow users to enable @truffle/db as system of record inside Truffle Feb 9, 2021
@gnidan gnidan changed the title Allow users to enable @truffle/db as system of record inside Truffle Allow users to enable @truffle/db as a system of record for their Truffle projects Feb 9, 2021
const OS = require("os");
const serveCommand = require("./commands/serve");

const usage =
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The style of these; does it really match with the rest of the Truffle help messages? Maybe we should indent "Available sub-commands" and "serve \tStart..." a bit more...or we could list the subcommands like we list the options? So they are indented far in, what do you think?

fainashalts and others added 24 commits February 24, 2021 16:38
So that networks and contract instances get saved in DB
Allow configuring host for `truffle db serve`
…-compile

Ensure artifacts are saved with db info
…cts (do not add abstract contracts or interfaces
Hook up `truffle migrate` to @truffle/db
Copy link
Contributor

@fainashalts fainashalts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Drumroll please... LGTM!

@gnidan gnidan merged commit 42c4adb into develop Feb 26, 2021
@gnidan gnidan deleted the truffle-db-mvp branch February 26, 2021 07:37
@gnidan
Copy link
Contributor Author

gnidan commented Feb 26, 2021

It's in :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants