Skip to content

NPM Workflows#114

Merged
elviskahoro merged 7 commits intowarpdotdev:mainfrom
webrune-tim:main
Aug 22, 2023
Merged

NPM Workflows#114
elviskahoro merged 7 commits intowarpdotdev:mainfrom
webrune-tim:main

Conversation

@webrune-tim
Copy link
Copy Markdown
Contributor

Discord username (optional) please include so we can attribute you with our Contributor role (like so elvis#4747)

Tithos

Description of changes (updated or new workflows)

I added a few NPM workflows for managing node_modules folders.

Copy link
Copy Markdown
Member

@alokedesai alokedesai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for contributing! Just a few small comments to clarify my understanding!

Comment thread specs/npm/look_for_all_directories_with_node_modules_and_show_them.yaml Outdated
Comment thread specs/npm/find_and_delete_node_modules_in_a_folder.yaml Outdated
Copy link
Copy Markdown
Member

@alokedesai alokedesai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One more comment! Besides that, it should be good to go :)

Comment thread specs/ssh/copy_ssh_key_for_github.yaml Outdated
Comment thread specs/ssh/generate_new_ssh_key_for_github.yaml Outdated
Copy link
Copy Markdown
Member

@alokedesai alokedesai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thank you for contributing!

@webrune-tim
Copy link
Copy Markdown
Contributor Author

webrune-tim commented Sep 30, 2022

@alokedesai Would you please add the `hacktoberfest-approved' label. Thank You

@alokedesai
Copy link
Copy Markdown
Member

@WebRuin Happy to add that label, but I believe we also need to opt-in the repo for hacktoberfest as well. Please correct me if I'm wrong

@alokedesai
Copy link
Copy Markdown
Member

Hey @WebRuin! Sorry for the late followup. Would you mind addressing any of the in-flight comments, would love to get all of these amazing workflows merged 🚢 Thanks again!

@elviskahoro elviskahoro self-requested a review August 22, 2023 09:18
@elviskahoro elviskahoro merged commit 564072e into warpdotdev:main Aug 22, 2023
elviskahoro pushed a commit that referenced this pull request Sep 5, 2023
* NPM Workflows

* github ssh

* npm updates

* npm fix

* github

* github

---------

Co-authored-by: Tim Smith <69870077+timscodebase@users.noreply.github.com>
Co-authored-by: elvis kahoro <elvis@warp.dev>
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

Successfully merging this pull request may close these issues.

4 participants