-
Notifications
You must be signed in to change notification settings - Fork 145
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
Suggestion: Tooling to assist maintenance errands #5
Comments
topic is too broad maybe? e.g. does I would suggest a narrow definition of what "core maintenance" needs are and short list tooling recommendation to match |
I guess it depends on how broad or specific this repo's initiative will be. For each best practice, there may be several automated tooling solutions, as well as there may be already some automated tooling solutions that people don't know about that could inspire a best practice. Ultimately, if it is a better fit outside this initiative, then an
Sure. eslint and prettier definitely help facilitate maintenance. |
|
Perhaps accompanied by a guide that focuses on workflow; how to use the tools most effectively. |
Linting to avoid bikeshedding also includes any usage of eslint, but especially the airbnb config. |
Automated tooling that creates PRs with package updates can go a long way too to preventing bugs and security issues from spreading too far (so long of course as the PRs are addressed in a timely manner). Ex (there are many such tools), this is just one I found googeling: https://greenkeeper.io/ Or even just add ing |
Would CodeClimate be a good link here: https://github.com/codeclimate/codeclimate? I've used this in similar tools to catch a good amount of basic code quality issues. If linting is not considered then nvm :P |
In addition to listing the tools, we may need to add pre-requisites. For example, if you add Greenkeeper you need to have some sort of CI tool enabled for it to work. If you use Renovate then does your project qualify as open source or is it a personal project and you have to pay after the trial period (it's not the amount that matters but the fact that you have to pay). Testing for bikeshedding depends on the tool you use and what preset is installed. We should encourage people to lint their code and provide alternatives rather than preach about single options. I also would like to see Eslint and its presets (Airbnb, Google, Standard) included in the linter section. Rather than bikeshedding, I would call it "code style consistency" or just plain linting. Last thought on this. I think it's important to note that these are options and we're not advocating specific ones. |
I saw this discussion as a great opportunity to discuss tooling that lowers the maintenance overhead. For example, I find it really helpful to maintain my packages using a dependency update tool. I feel it would be really helpful to maintainers to suggest tools (and not a particular brand) that can help them maintain their projects and how it does so (how it can help them and the community) |
Maybe someone could create an awesome list and we could say to users:
? |
copy of my comment here #280 (comment) FWIW @bevry accomplishes the goal of this automatically for all of its packages, through the following pieces
|
@balupton sounds like a cool set of tools! I think in the existing docs we have avoided making specific package recommendations, but instead we list several packages that do similar things. It would probably be useful to mention some of your tools in some of our existing guidelines, as well as providing alternatives? |
Sure, that's your discretion. |
@balupton, if you can identify places where your tools could be mentioned in the documentation we are building, it would be helpful to open PR's against those so we are sure to include them in the right places. I don't think those here will always know which of the ones you list are intended to fit where in the SDLC. |
@wesleytodd seems the only page would be https://github.com/nodejs/package-maintenance/raw/master/docs/drafts/tools.md |
Great! The PRs should be simple enough then. Can we close this issue? |
Travis CI recently informed me that I have over 800 packages using it.
I've done up the following tooling to assist in automating the maintenance of my packages as much as possible, they have been used for years, and probably others will find them useful:
If you have your own, post them too.
The text was updated successfully, but these errors were encountered: