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

Analizzi scripts to rule them all #34

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

analizzi
Copy link
Collaborator

@analizzi analizzi commented May 6, 2023

Overview

Creating update/bootstrap script and a server script

Closes #5
Closes #17

Demo

Paste a screenshot or terminal output. Before and after is your and your reviewer's friend

Testing instructions

  • I dont know how to test this
  • Maybe try running the scripts?

Checklist before merge

  • Update the CHANGELOG.md
  • Rebase your changes from whatever branch you're merging to

@travissouthard travissouthard mentioned this pull request May 6, 2023
3 tasks
@travissouthard travissouthard self-requested a review May 6, 2023 13:04
@travissouthard travissouthard self-assigned this May 6, 2023
scripts/update Outdated Show resolved Hide resolved
@analizzi analizzi force-pushed the analizzi-Scripts_to_rule_them_all branch from 7630ab3 to 2fdeaa3 Compare May 7, 2023 16:08
@analizzi analizzi requested a review from mdarkcity May 7, 2023 16:10
@travissouthard travissouthard removed their request for review May 7, 2023 16:10
`git update-index --chmod=+x scripts/*`
@mdarkcity
Copy link
Collaborator

mdarkcity commented May 7, 2023

I pushed a commit to set the script permissions so that they'd be executable when others pull them down:
git update-index --chmod=+x scripts/*

scripts/bootstrap Outdated Show resolved Hide resolved
@analizzi analizzi requested a review from mdarkcity May 7, 2023 18:00
@mdarkcity
Copy link
Collaborator

mdarkcity commented May 7, 2023

Feel free to update CHANGELOG.md here before merging, as in #33 (pointing to this PR)

- Bootstrap/Update script and Server script [#34](https://github.com/CodeForPhilly/third-places/pull/34)

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.

Update requirement passage in README Add scripts based on "Scripts to rule them all" format
3 participants