Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Related Project #1

Closed
MylesBorins opened this issue Oct 24, 2017 · 23 comments
Closed

Related Project #1

MylesBorins opened this issue Oct 24, 2017 · 23 comments

Comments

@MylesBorins
Copy link
Member

MylesBorins commented Oct 24, 2017

Taken from nodejs/TSC#392 (comment)

@joyeecheung
Copy link
Member

Also I am working on https://github.com/joyeecheung/node-core-utils, it can be transferred when it's ready (right now changes too fast and too much force-pushing lol)

@MylesBorins
Copy link
Member Author

@joyeecheung please feel free to add to the original post

(same to anyone else reading this)

@refack
Copy link

refack commented Oct 24, 2017

I've thrown the name GYP around... It's not an obvious fit, but this SIG is the closest home currently available.

@refack
Copy link

refack commented Oct 24, 2017

P.S. Using the project boards (in addition to labels) seems like a good fit.

@maclover7
Copy link

maclover7 commented Oct 24, 2017

To at least start off, would it make sense to structure nodejs/automation as a Lerna repo, and import branch-diff, changelog-maker, and core-validate-commit that way? So that way at least all of the modules are in one place, and then we can try and refactor/go from there.

@joyeecheung
Copy link
Member

@maclover7 Hmmm..what do we gain by moving their source code into one repository tho? If we want to distribute them together we can just make a package that set up multiple bin entries..

@gibfahn
Copy link
Member

gibfahn commented Oct 24, 2017

I guess the downside to that is that we'd lose commit history for the existing repos too.

@joyeecheung
Copy link
Member

I've thrown the name GYP around... It's not an obvious fit, but this SIG is the closest home currently available.

@refack Hmm..do we still plan to use GYP, like, forever..? I guess at some point we might have to switch to GN? If so I guess it makes more sense to focus on the transition (although that sounds more like core stuff or build stuff)

@maclover7
Copy link

maclover7 commented Oct 24, 2017

Hmmm..what do we gain by moving their source code into one repository tho?

@joyeecheung My thought was that it's easier to keep all of the tools automation might be maintaining in a single place. Also, if some tools might be combined/sharing logic, it would be easier to keep everything together (PRs could affect multiple tools, easier to review via one repo).

I guess the downside to that is that we'd lose commit history for the existing repos too.

@gibfahn I have to refresh my git-fu, but I think we can do a "subtree merge" to keep all of the existing history

@refack
Copy link

refack commented Oct 24, 2017

@refack Hmm..do we still plan to use GYP, like, forever..? I guess at some point we might have to switch to GN? If so I guess it makes more sense to focus on the transition (although that sounds more like core stuff or build stuff)

Just for context (still not sure if this is the proper home)

  1. GYP IMHO is a great tool. BTW even outside of the node ecosystem it's more popular than gn
  2. We should start maintaining it properly, i.e. new platform & bug fixes
  3. The "optimal" transition IMHO is to gyp.js, not gn (V8 building issue has a resolution plan)
  4. That's because of the substantial ecosystem investment in addon build scaffolding via node-gyp
  5. P.S. node-gyp

@refack
Copy link

refack commented Oct 24, 2017

I guess the downside to that is that we'd lose commit history for the existing repos too.

There's a way to graft git trees together, so we'll only lose the GitHub meta history. I'm also not crazy about adding 7-10 repos.
Ref: Monorepos in the wild etc.

@joyeecheung
Copy link
Member

Just for context (still not sure if this is the proper home) ...

@refack Thanks for the context (and I think this is probably not something that should be handed over to the automation team :S...this is related to how the core is built & how addons are built so deserves its own taskforce)

@refack
Copy link

refack commented Oct 26, 2017

Another tool https://github.com/nodejs/nodejs-latest-linker - ATM it does not create a link for latest nightly nor latest v8-canary.
Refs: nodejs/build#908

@Tiriel
Copy link
Contributor

Tiriel commented Oct 31, 2017

Just to keep moving, where do we get from here?

I do like the idea of a monorepo, but how do we really implement it and transfer repos from various places to here?

@joyeecheung
Copy link
Member

@maclover7 mentioned Lerna (#1 (comment)), I think this looks like a good tool to get things started?

@Tiriel
Copy link
Contributor

Tiriel commented Oct 31, 2017

LGTM!

@maclover7
Copy link

👍 whenever nodejs/TSC/issues/406 is approved, I'll start moving stuff in :)

@refack
Copy link

refack commented Nov 1, 2017

👍 whenever nodejs/TSC#406 is approved, I'll start moving stuff in :)

I have a snippet that does git tree grafting.

@Tiriel
Copy link
Contributor

Tiriel commented Nov 1, 2017

Just wanted to say: I am deeply excited to be part of all of this.
More than I care to admit in fact.

This is just awesome!

@Tiriel
Copy link
Contributor

Tiriel commented Nov 10, 2017

Can't edit issues on this repo (yet?) but the link for node-core-utils should be updated in the issue, still points toward the old repo.

@rvagg
Copy link
Member

rvagg commented Nov 11, 2017

FYI my node tools have grown out of this repo https://github.com/rvagg/iojs-tools, there's still a few in there you can have if anyone cares. pr-metadata is something I've still found helpful but the lack of a GitHub API for the review feature has made it less useful (maybe there's an API or we'll get one soon). Evan's Chrome extension seems to do a more complete job for now.

@joyeecheung
Copy link
Member

joyeecheung commented Nov 11, 2017

@rvagg By "GitHub API for the review feature" do you mean "GitHub API for getting reviewers and their review status"? If so that's available now and is being used by get-metadata, which is a CLI alternative to node-review. The GraphQL query we use to get the reviews is here.

@rvagg
Copy link
Member

rvagg commented Nov 12, 2017

Cool, wasn't even available on GraphQL last time I looked. If you have a new utility that does this work then that's great and you don't need mine!

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

No branches or pull requests

7 participants