fig - move to another repo

[dmouraneto]

Hi Greg,

Did you consider moving fig to another repo?
If someone else wants to just use fig but not your aspects it would be easier.

[wincent]

It's one of those things where there is no point in moving it until there is demand to do so. But we can count your issue as the first evidence that it might be worth doing. 😁

FWIW, there are three main options:

• If you want to use Fig without the aspects, you can fork the repo and delete the aspects. Whenever Fig updates (which should be rare... only for bug fixes or when I discover I need a new feature), you could get that via git fetch and merging in the changes.
• Extract Fig into a separate repo: it could be an npm package or something that you could add to another repo via Git's submodule mechanism (the latter is what I would do if I was consuming it from my dotfiles repo, as I already rely heavily on submodules to pull in other stuff).
• Extract Fig into a "template" repo, which would basically be a boilerplate that you could clone or fork with no aspects, and then add your own. Again the update process here would involve a fetch + a merge.

My inclination is to leave this open for a while to see if anybody else has anything to add.

[dmouraneto]

I'm using via Git's submodule mechanism.
But I had to create symlinks to bin, fig, support and vendor.
And copied the ./install.

[wincent]

Are your dotfiles up somewhere public? I always like to see other people's set-ups...

[dmouraneto]

I didn't have my dotfiles organized and public, now you asked I published then dmouraneto/dmouraneto
I just recently started to put some effort on that, I watched your talk at vim.conf and inspired me a lot, thank you for that.

I made some local changes to your code that isn't in github (I have to fix that). The change I made was on function loadAspect in fig/main.ts to

async function loadAspect(aspect: Aspect | string): Promise<void> {
   try {
       await import(`../aspects/${aspect}/index.js`);
   } catch (e) {
       throw new Error(`Unreachable ${aspect}`);
    }
}

The way it was before it was not working for me.