Skip to content
This repository was archived by the owner on Jan 16, 2025. It is now read-only.

Updated readme to fix broken links with alternatives in the README and made some script changes (See Description) #1

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

tristanpoland
Copy link
Member

@tristanpoland tristanpoland commented Sep 4, 2024

Enhance setup process with dependency checks and user prompts

Summary

This PR improves the setup process for our pipeline templates repo by adding dependency checks for fly (Concourse CLI) and bosh CLI, and introducing user prompts for installation. These changes make the setup process more robust and user-friendly.

Changes

  1. Created a new ensure_deps.sh script to check and optionally install dependencies.
  2. Modified the main setup script to use the new dependency check.
  3. Added user prompts before installing any missing dependencies.
  4. Improved error handling and made the setup process more resilient to missing dependencies.

Details

New ensure_deps.sh script

  • Checks for the presence of fly and bosh CLIs.
  • Prompts the user before attempting to install any missing dependency.
  • Supports multiple Linux distributions (Debian, Red Hat, Arch) and macOS.
  • Provides clear feedback on the installation status of each dependency.

Updates to setup script

  • Calls the new ensure_deps.sh script to check dependencies.
  • Warns the user if dependencies are missing and allows them to proceed and attempt to install them or abort.
  • Improved handling of cases where fly or bosh commands might not be available.

Testing

  • Tested the scripts on Debian-based Linux (in WSL)
  • Verified behavior when dependencies are present, and missing

Appended Updates

There are now updates on this PR bringing it to the latest golden sample version we have

@tristanpoland tristanpoland changed the title Updated readme to fix broken links with alternatives and made some script changes (See Description) Updated readme to fix broken links with alternatives in the README and made some script changes (See Description) Sep 4, 2024
Note for @dennisjbell We will change /envs/* later to make a more general example
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant