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

Added possibility to check on :priority in test helper methods #41127

Merged
merged 1 commit into from
Jul 15, 2021

Conversation

morgoth
Copy link
Member

@morgoth morgoth commented Jan 15, 2021

Retrying #26698

@rails-bot rails-bot bot added the activejob label Jan 15, 2021
@rails-bot
Copy link

rails-bot bot commented Apr 15, 2021

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
Thank you for your contributions.

@morgoth
Copy link
Member Author

morgoth commented Apr 15, 2021

This is still valid PR. I rebased to resolve conflicts

@rails-bot
Copy link

rails-bot bot commented Jul 14, 2021

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
Thank you for your contributions.

@rails-bot rails-bot bot added the stale label Jul 14, 2021
@morgoth
Copy link
Member Author

morgoth commented Jul 14, 2021

This is still valid

@rails-bot rails-bot bot removed the stale label Jul 14, 2021
Copy link
Member

@eugeneius eugeneius left a comment

Choose a reason for hiding this comment

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

This looks good to me, would you mind resolving the conflict?

@morgoth
Copy link
Member Author

morgoth commented Jul 15, 2021

@eugenebolshakov rebased.

@eugeneius eugeneius merged commit ebebb9d into rails:main Jul 15, 2021
@eugeneius
Copy link
Member

Thanks!

@morgoth morgoth deleted the test-job-priority branch July 15, 2021 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants