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

Create a contributor bootstrap script #3028

Closed
sherifnada opened this issue Apr 22, 2021 · 7 comments
Closed

Create a contributor bootstrap script #3028

sherifnada opened this issue Apr 22, 2021 · 7 comments
Assignees
Labels
accepting-contributions Feel free to contribute to them team/platform-move team/prod-eng type/enhancement New feature or request

Comments

@sherifnada
Copy link
Contributor

sherifnada commented Apr 22, 2021

Tell us about the problem you're trying to solve

As a contributor I want to run a single bash command that verifies that I have all the necessary commands to get started:

  • Docker
  • Java 14
  • pip 20.1
  • Python 3.7.9
  • jq

Describe the solution you’d like

A single bash script that makes this process as seamless as possible by verifying that I have the right version of all software installed.

Describe the alternative you’ve considered or used

Additional context

┆Issue is synchronized with this Asana task by Unito

@sherifnada sherifnada added type/enhancement New feature or request accepting-contributions Feel free to contribute to them good first issue labels Apr 22, 2021
@nclsbayona
Copy link
Contributor

Hi! Not sure if this is what you meant, maybe it is, please let me know.
check_requirements.txt

@sherifnada
Copy link
Contributor Author

@nclsbayona thanks for creating this! This is a great start. some comments:

  1. Could you create a PR so we can review/merge afterwards?
  2. could you put this into a bash script in ./tools/bin/ ?

@nclsbayona
Copy link
Contributor

nclsbayona commented Apr 26, 2021

I made PR #3054

cgardens pushed a commit that referenced this issue Apr 26, 2021
@cgardens
Copy link
Contributor

@sherifnada i merged @nclsbayona 's script. Do we want to add it as part of the build or should we just add it to the docs as a pre requ?

@sherifnada
Copy link
Contributor Author

We should definitely add it to the getting started docs and maybe to the build if we can get any error messages to display nicely. Will close this ticket once that's done.

@noahkawasakigoogle
Copy link
Contributor

Related PR: #9230

@supertopher
Copy link
Contributor

supertopher commented Feb 22, 2023

I believe that we have solved this problem through this PR

if you disagree please come in. closing the issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepting-contributions Feel free to contribute to them team/platform-move team/prod-eng type/enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

7 participants