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

Upstream 0.25.0 #1

Merged
merged 32 commits into from
May 13, 2024
Merged

Upstream 0.25.0 #1

merged 32 commits into from
May 13, 2024

Conversation

o-orand
Copy link
Member

@o-orand o-orand commented May 13, 2024

bgandon and others added 30 commits March 6, 2024 15:29
Properly using build args as advised in the docs.
New metadata file that supports the new 'load_var' Concourse statement
Add PR body in the metadata provided by 'get' steps
Indeed, in order to delete the correct comments, the golang code makes sure
that comment's `Author.Login` is the same as current `Viewer.Login`. But with
bot users, viewer's login is "githubApp[bot]" and author login is "githubApp".

As a consequence, for the comparison to succeed, we need to remove any
trailing "[bot]" suffix from both parties, and compare the resulting prefixes.

This isue was originally submitted as telia-oss#245 and
fixed later in monzo#4.
@o-orand o-orand merged commit 2f74a8d into orange-main May 13, 2024
0 of 2 checks 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.

4 participants