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

fix: 6205 follow-up #6206

Merged
merged 2 commits into from
Aug 13, 2024
Merged

Conversation

UdjinM6
Copy link

@UdjinM6 UdjinM6 commented Aug 12, 2024

Issue being fixed or feature implemented

I think I messed up whitespaces so these jobs can't run now, they fail with [Error] Invalid type for on https://github.com/dashpay/dash/actions/runs/10353281487...
Weird that they succeeded in #6205 https://github.com/dashpay/dash/actions/runs/10352521500/workflow https://github.com/dashpay/dash/actions/runs/10352521490/workflow 🤷‍♂️

What was done?

How Has This Been Tested?

Breaking Changes

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation
  • I have assigned this pull request to a milestone (for repository code-owners and collaborators only)

@UdjinM6
Copy link
Author

UdjinM6 commented Aug 12, 2024

Hmm.. or maybe there smth else

@UdjinM6
Copy link
Author

UdjinM6 commented Aug 12, 2024

For whatever reason GH refuses to even try to start "check merge ff" on dashpay for me (happened a couple of times in earlier PRs too), not sure why it happens or how to fix it. Runs with no issues in my repo https://github.com/UdjinM6/dash/actions/runs/10354130145.

@PastaPastaPasta
Copy link
Member

SOooo are we good to merge here?

@UdjinM6
Copy link
Author

UdjinM6 commented Aug 13, 2024

Let's merge and find out :D

Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

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

utACK 52b9fce

@PastaPastaPasta PastaPastaPasta merged commit 792c032 into dashpay:develop Aug 13, 2024
16 of 17 checks passed
@UdjinM6 UdjinM6 added this to the 21.2 milestone Aug 23, 2024
@UdjinM6 UdjinM6 modified the milestones: 21.2, 22 Oct 29, 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.

2 participants