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.

Sign up for GitHub

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

Jump to bottom

Update dynamic README workflow to open a Pull Request #31

Merged
merged 4 commits into from
Apr 10, 2024
Merged

Update dynamic README workflow to open a Pull Request #31

merged 4 commits into from
Apr 10, 2024

Commits on Apr 2, 2024

  1. Update dynamic render workflow 

    Some repositories have a protected branch rule. That means
our workflow won't work because commiting directly to main
is not allowed. To prevent that, the workflow will now
open a PR for the changes.
    Stefanni Brasil committed Apr 2, 2024
    Configuration menu
    Copy the full SHA
    961d0af View commit details
    Browse the repository at this point in the history

  2. Update workflow example 

    We only want to run this workflow if the README has changed, not
on every change.
    Stefanni Brasil committed Apr 2, 2024
    Configuration menu
    Copy the full SHA
    f41da90 View commit details
    Browse the repository at this point in the history

  3. Removing leading / from snippet 

    As @nickcharlton pointed out,

> the leading / implies to me the start of a filesystem,
which it presumably isn't.

I tested removing it and it worked, so it was probably
needed when I was testing things but it's not needed anymore.
    Stefanni Brasil committed Apr 2, 2024
    Configuration menu
    Copy the full SHA
    8464adf View commit details
    Browse the repository at this point in the history

  4. Use latest workflow version

    Stefanni Brasil committed Apr 2, 2024
    Configuration menu
    Copy the full SHA
    d2c7270 View commit details
    Browse the repository at this point in the history