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

Looking for issue good for first contribution #495

Closed
piccoloaiutante opened this issue Sep 20, 2016 · 15 comments
Closed

Looking for issue good for first contribution #495

piccoloaiutante opened this issue Sep 20, 2016 · 15 comments

Comments

@piccoloaiutante
Copy link
Member

Hi @mhdawson,

we met at NodeInteractive and as I told you I would like to start contributing to this working group. Is there any issue that is a good starting point for someone that is doing his first contribution?

Thanks

@mhdawson
Copy link
Member

Sorry been catching up after being away at Node Interactive. Will try to come up with a good suggestion this week.

@Fishrock123
Copy link
Contributor

ping @mhdawson

@mhdawson
Copy link
Member

@Fishrock123 thanks for the reminder, will bring it up in WG meeting to day to see what kind of suggestions I can come up with. @piccoloaiutante you may also want to join the meeting today to listen to get a feel with respect to the issues being discussed/worked.

@jbergstroem
Copy link
Member

I guess it depends on what the interest might be. I can see a few things between this and the github-bot group.

@piccoloaiutante
Copy link
Member Author

piccoloaiutante commented Oct 11, 2016

@mhdawson @jbergstroem i can join you in WG meeting in half hour so we can discuss about it

@jbergstroem
Copy link
Member

@piccoloaiutante lets keep the discussion post the meeting. I can stick around a bit longer and I'm sure @mhdawson can too! You're still welcome to join in and listen to our meeting.

@piccoloaiutante
Copy link
Member Author

@jbergstroem no problem, i'll follow it. Then i'm not sure that i'll stick around after it since i'm based in italy, but anyway we can go on with this issue.

@joaocgreis
Copy link
Member

Welcome @piccoloaiutante ! Here are some ideas:

  • Adapt the Windows Ansible scripts to the refactor by Johan, integrating Windows as well as possible. Some things that would be nice to have:
    • Handling reboots, when updating Windows or installing Visual Studio.
    • Updating Windows before starting to install programs.
    • Generating .rdp and/or .remmina files (I have a rough script for Remmina that can serve as a starting point).
    • Handle Windows secrets better - passwords are used only for Windows and currently the file we have is not parsable, so we can decide for something better that Ansible can pick from the secrets (currently I have a custom script to generate host_vars).
  • vcbuild.bat support for Expand coverage of v8 testing from Node.js tree #387 , porting what is currently in the Makefile (introduced initially in test: run v8 tests node#4704 ).

@piccoloaiutante
Copy link
Member Author

piccoloaiutante commented Nov 7, 2016

@joaocgreis I have a first draft of ansible script for supporting widows update and reboot. This should cover your first two points. I tested it on windows 10 and windows server 2012. Can you have a quick look at it before I submit a PR for it just to see that I'm on track of what you're expecting?

https://gist.github.com/piccoloaiutante/d6f0766235b63d44defd3d35bc1887c7

@piccoloaiutante
Copy link
Member Author

@joaocgreis i can get it Windows Server 2008R2 from msdn. Don't worry i'll be back with info about it. Thanks

@piccoloaiutante
Copy link
Member Author

@joaocgreis i spent a lot of time trying to install power shell 3.0 on windows 2008 R2 so i could run the setup script but i didn't succeed. Is there anything particular that i should do for provisioning this OS?

@joaocgreis
Copy link
Member

joaocgreis commented Nov 21, 2016

@piccoloaiutante I've used the script at http://docs.ansible.com/ansible/intro_windows.html#getting-to-powershell-3-0-or-higher before, it worked for me. (Feel free to move this conversation to IRC or email, I don't mind interacting more to help you)

@joaocgreis
Copy link
Member

I also had issues updating PowerShell in my local VM, the script didn't work. But the instructions at https://msdn.microsoft.com/en-us/powershell/scripting/setup/installing-windows-powershell#a-namebkmkinstallingonwindows7andwindowsserver2008r2ainstalling-windows-powershell-on-windows-7-and-windows-server-2008-r2 did it.

@joaocgreis
Copy link
Member

As noted by @nschonni in #750 (comment) , we don't have a CONTRIBUTING.md. We could add one pointing to https://github.com/nodejs/node/blob/master/CONTRIBUTING.md , with also a placeholder to mention any differences or additions. Furthermore, the templates (https://github.com/nodejs/node/tree/master/.github) would also be good to have. This could probably be a good first contribution.

@mhdawson
Copy link
Member

mhdawson commented Sep 8, 2017

@piccoloaiutante I think we can close this now as you are part of the WG :)

@mhdawson mhdawson closed this as completed Sep 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants