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

Set PR_NUMBER when PR is created #114

Merged
merged 1 commit into from
Feb 12, 2023

Conversation

GlacierWalrus
Copy link
Contributor

Fixes #104 (comment)

pr_number output is unset if this job creates the PR, add a gh command to retrieve the number based on the url returned by creation command.

💻 Commits

📁 Modified files

⚠️ Additional information

  • Pushed to a branch with a proper name and provided proper commit message.
  • [ x ] Provided a clear and concise description of what the issue is.

Check CONTRIBUTING.md and CODE_OF_CONDUCT.md for more information

@jit-ci
Copy link

jit-ci bot commented Feb 10, 2023

Hi, I’m Jit, a friendly security platform designed to help developers build secure applications from day zero with an MVS (Minimal viable security) mindset.

In case there are security findings, they will be communicated to you as a comment inside the PR.

Hope you’ll enjoy using Jit.

Questions? Comments? Want to learn more? Get in touch with us.

@ChristophShyper ChristophShyper self-assigned this Feb 11, 2023
ChristophShyper added a commit that referenced this pull request Feb 12, 2023
Preparation to merge #114
@github-actions github-actions bot mentioned this pull request Feb 12, 2023
2 tasks
@ChristophShyper ChristophShyper changed the base branch from master to bugfix/pr-number February 12, 2023 09:34
@ChristophShyper ChristophShyper added the bugfix Bug needing a fix label Feb 12, 2023
@ChristophShyper ChristophShyper merged commit a6bd1d2 into devops-infra:bugfix/pr-number Feb 12, 2023
@ChristophShyper
Copy link
Member

@GlacierWalrus Thank you for the contribution.
gh wasn't installed in the image. hub was. But now I've added the missing one.
It has been released in devops-infra/[email protected].

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Bug needing a fix
Development

Successfully merging this pull request may close these issues.

Add pull request number as an output
2 participants