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

chore: remove danger and jenkins related files #1368

Merged
merged 2 commits into from
Jan 8, 2024
Merged

Conversation

ewanharris
Copy link
Contributor

@ewanharris ewanharris commented Jan 8, 2024

These are no longer used

Going to repurpose this into general npm updates due to slow build times

@ewanharris ewanharris requested a review from m1ga January 8, 2024 20:53
@m1ga
Copy link
Contributor

m1ga commented Jan 8, 2024

Looks fine. local tests are still running but no error so far (it's a devDep anyways). Can you disable dependabot too? don't like all the auto PRs 😄

Copy link
Contributor

@m1ga m1ga left a comment

Choose a reason for hiding this comment

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

LGTM

@ewanharris ewanharris merged commit 02feed8 into master Jan 8, 2024
5 checks passed
@ewanharris ewanharris deleted the deps/remove-danger branch January 8, 2024 21:24
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