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

Create Pull Request #1627

Closed
wants to merge 3 commits into from
Closed

Conversation

validcube
Copy link
Contributor

@validcube validcube commented Dec 5, 2023

Description

Create a Pull Request instead of pushing directly to repository (See relevant rule: GH006).

Fixes #1626

Type of change

✅ Bug fix (non-breaking change which fixes an issue)
❌ General change (non-breaking change that doesn't fit the above categories, such as copyediting)

Copy link
Contributor Author

@validcube validcube left a comment

Choose a reason for hiding this comment

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

bleep: self-review

pull_request:

jobs:
optimize-auto-merge:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
optimize-auto-merge:
merge-optimize-svg:

jobs:
optimize-auto-merge:
runs-on: ubuntu-latest
if: github.event.pull_request.title == 'Optimize SVG(s)'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Checking only for the title is not ideal

See https://github.com/LawnchairLauncher/lawnchair/blob/13-dev/.github/workflows/ci.yml#L109-L119 for a better implementation

PR_URL: ${{ github.event.pull_request.html_url }}
run: |
gh pr comment $PR_URL --body "No issues found, auto-merging the pull request."
gh pr merge $PR_URL --auto --squash --delete-branch
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Might be a problem, because we would need to let it run a bit for the required check to complete then we can merge it.

@validcube validcube closed this Mar 30, 2024
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.

[DISCUSSION] Workflow to optimize SVGs is not working
1 participant