-
Notifications
You must be signed in to change notification settings - Fork 21
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
Adding automatic PR creation for generated README.md & installers.toml #69
Adding automatic PR creation for generated README.md & installers.toml #69
Conversation
5502849
to
880fe52
Compare
With out the / this gets installers.toml also
run: | | ||
pip install toml pytablewriter | ||
python generate.py ${{ env.GIT_DIFF }} | ||
docker build -t shellspec . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's move this Docker image to GitHub Packages and put up a GitHub Action to build it and push. We can simply pull the image here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's take this into this week #59
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks ok to me.
…rs-toml-69 Update installer scripts, README.md & installers.toml for #69
With this PR, the Continuous Delivery part of the installers completes. Now when a someone creates a PR with a new
installer.toml
file for a tool and gets it merged, the GitHub Actions will generate theinstaller.sh
,installer.min.sh
then updates theREADME.md
andinstallers.toml
with a summary.Sample PR from someone: Heshdude#43 ( only
installer.toml
file )Auto generated PR: Heshdude#44 (
installer.sh
,installer.min.sh
,README.md
andinstallers.toml
)