Skip to content

Conversation

@SorchaAbel
Copy link
Contributor

@SorchaAbel SorchaAbel commented Jun 24, 2025

update bundler to 2.4.22
public pipeline

@SorchaAbel SorchaAbel requested review from a team and sj26 June 24, 2025 03:59
Readme.md Outdated

This repository is an example on how to easily run 20 parallel testing agents for a [Rails](https://rubyonrails.org/) application using [Buildkite](https://buildkite.com/), [rbenv](https://github.com/rbenv/rbenv) and [Knapsack](https://github.com/ArturT/knapsack).

See the full [Getting Started Guide](https://buildkite.com/docs/guides/getting-started) for step-by-step instructions on how to get this running, or [Add to Buildkite](https://buildkite.com/new) to try it yourself.Add commentMore actions
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
See the full [Getting Started Guide](https://buildkite.com/docs/guides/getting-started) for step-by-step instructions on how to get this running, or [Add to Buildkite](https://buildkite.com/new) to try it yourself.Add commentMore actions
See the full [Getting Started Guide](https://buildkite.com/docs/guides/getting-started) for step-by-step instructions on how to get this running, or [Add to Buildkite](https://buildkite.com/new) to try it yourself.

Copy link
Contributor

@123sarahj123 123sarahj123 Jun 24, 2025

Choose a reason for hiding this comment

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

Also have added two buttons "Add to Buildkite" - not sure if this was what you were after :)

Copy link
Contributor

Choose a reason for hiding this comment

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

image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

2 buttons is fine, one it to run it in your own pipeline and the other is public. All the repos have it 👍

Copy link
Contributor

Choose a reason for hiding this comment

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

sweet, thanks for clearing up :)

@SorchaAbel SorchaAbel force-pushed the PB-39-rails-parallel branch from 7574b1c to 296450b Compare June 24, 2025 04:51
@SorchaAbel SorchaAbel requested a review from 123sarahj123 June 24, 2025 04:55
Copy link
Contributor

@123sarahj123 123sarahj123 left a comment

Choose a reason for hiding this comment

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

LGTM :)

@SorchaAbel SorchaAbel merged commit 53f3fcd into main Jun 24, 2025
1 check passed
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.

3 participants