Skip to content

Replace deprecated Concourse resource for pull requests #19944

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

Conversation

dreis2211
Copy link
Contributor

Hi,

apparently https://github.com/jtarchie/github-pullrequest-resource has been replaced by https://github.com/telia-oss/github-pr-resource since the end of 2018 already.

I don't know if this is a problem for the Concourse setup when we switch resources. I remember concourse/concourse#145 but that should be solved.

Let me know what you think.
Cheers,
Christoph

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jan 27, 2020
@snicoll snicoll self-assigned this Jan 28, 2020
@snicoll snicoll added type: task A general task and removed status: waiting-for-triage An issue we've not yet triaged labels Jan 28, 2020
@snicoll snicoll added this to the 2.1.13 milestone Jan 28, 2020
@snicoll
Copy link
Member

snicoll commented Jan 28, 2020

I've update the pipeline on master with those changes. Let's see what happens on the next PR :)

@dreis2211
Copy link
Contributor Author

So you updated the pipeline to my branch? I'm a bit confused because I see no commit 😉

@snicoll
Copy link
Member

snicoll commented Jan 28, 2020

nope. I updated our pipeline with this PR. I don't have to merge the PR to make that happen. For some reason, building PRs is lagging behind quite a lot, it's not done yet with your Asciidoctor 3.0 PR :/

This PR does not have a CI check (new since the change): #19959

@dreis2211
Copy link
Contributor Author

Good to know that this doesn't need merging to apply it :)

@snicoll
Copy link
Member

snicoll commented Jan 28, 2020

Alright the CI check is there now so looks like everything is working as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: task A general task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants