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

refactor(createGHA): drop hercule in favor of native solution #660

Merged
merged 2 commits into from
Nov 3, 2022

Conversation

kanadgupta
Copy link
Member

@kanadgupta kanadgupta commented Nov 3, 2022

🧰 Changes

As part of our woes with the sindr*sorhus industrial complex, this PR removes hercule in favor of a native solution using String.replace() that was stupidly clean and simple.

Partially fixes #661.

🧬 QA & Testing

We have a ton of comprehensive snapshot testing for our GitHub Actions workflow files, so if those tests pass, we should be good to go.

@kanadgupta kanadgupta added dependencies Pull requests that update a dependency file refactor Issues about tackling technical debt labels Nov 3, 2022
@kanadgupta kanadgupta requested a review from erunion November 3, 2022 01:00
@kanadgupta kanadgupta marked this pull request as ready for review November 3, 2022 01:00
Copy link
Member

@erunion erunion left a comment

Choose a reason for hiding this comment

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

🚮

@kanadgupta kanadgupta merged commit e55b89e into main Nov 3, 2022
@kanadgupta kanadgupta deleted the drop-hercule branch November 3, 2022 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file refactor Issues about tackling technical debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TypeScript build error when rdme@8 is included in deps
2 participants