Skip to content

feat: improve celestia-app.sh script use#1243

Merged
jcstein merged 1 commit intomainfrom
jcs/update-app-script
Nov 3, 2023
Merged

feat: improve celestia-app.sh script use#1243
jcstein merged 1 commit intomainfrom
jcs/update-app-script

Conversation

@jcstein
Copy link
Copy Markdown
Member

@jcstein jcstein commented Nov 3, 2023

Overview

This improves the UX for use of the celestia-app.sh script, with a suggestion from @ramin. This eliminates the need for the user to download the script.

Checklist

  • New and updated code has appropriate documentation
  • New and updated code has new and/or updated testing
  • Required CI checks are passing
  • Visual proof for any user facing features like CLI or documentation updates
  • Linked issues closed with keywords

@jcstein jcstein self-assigned this Nov 3, 2023
@jcstein jcstein requested review from ramin and rootulp November 3, 2023 13:01
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 3, 2023

PR Preview Action v1.4.4
🚀 Deployed preview to https://celestiaorg.github.io/docs/pr-preview/pr-1243/
on branch gh-pages at 2023-11-03 13:02 UTC

Copy link
Copy Markdown
Contributor

@ramin ramin left a comment

Choose a reason for hiding this comment

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

clearly a work of genius

@jcstein jcstein merged commit a124684 into main Nov 3, 2023
@jcstein jcstein deleted the jcs/update-app-script branch November 3, 2023 13:15
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.

2 participants