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

Add remark deps #17315

Closed
wants to merge 2 commits into from
Closed

Conversation

MylesBorins
Copy link
Contributor

I was unable to reopen #16635 so I am alternatively opening this new PR

Since #16635 was closed we have noticed weird edge cases in multiple code and learns with random artifacts being created in peoples git repositories... this is making it harder for people to submit changes to Node

example of a PR with extra files: #17306
example of a PR to fix the gitignore issue: #17224

@refack blocked on the original PR due to filesize, if they (or anyone else) cannot come up with an alternative PR before the TSC meeting on Wednesday I'm going to escalate this to the TSC to reach consensus on. This is causing very real friction for developers and we should prioritize fixing this ASAP

@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. tools Issues and PRs related to the tools directory. labels Nov 26, 2017
@MylesBorins MylesBorins added the tsc-agenda Issues and PRs to discuss during the meetings of the TSC. label Nov 26, 2017
@MylesBorins
Copy link
Contributor Author

MylesBorins commented Nov 26, 2017

It is worth mentioning that if we cannot find a solution to this an alternative will be backing out remark until we can come up with a proper solution.

@targos
Copy link
Member

targos commented Nov 26, 2017

Did you run dmn like we do for ESLint to reduce the number of files?

dmn -f clean

@MylesBorins
Copy link
Contributor Author

@targos this was a 1:1 recreation of #16635

I'm putting together an alternative PR right now that makes an update-remark script using dmn. also looking into including npm dedupe as well. We may want to examine doing this for eslint too

@MylesBorins
Copy link
Contributor Author

Closing as dupe

@Trott Trott removed the tsc-agenda Issues and PRs to discuss during the meetings of the TSC. label Dec 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Issues and PRs related to build files or the CI. tools Issues and PRs related to the tools directory.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants