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.

Sign up for GitHub

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

Jump to bottom

Get the PR info by ID or associated commit #12

Merged
merged 9 commits into from
Sep 5, 2024
Merged

Get the PR info by ID or associated commit #12

merged 9 commits into from
Sep 5, 2024

Conversation

goruha
Copy link

@goruha goruha commented Aug 28, 2024 •
edited
Loading

What

  • Extend GHA to support retrieving PR by ID
  • Replaced usage of npm with yarn
  • Added tests for getting PR by PR ID
  • Copy package.json and jest configs from cloudposse typescript actions

Why

  • Working with issue comments, we have PR ID instead of commit SHA in the context.
  • Standardazise toolchain
  • Improve test coverage
  • Standardazise config and workflows

References

@goruha goruha added the major Breaking changes (or first stable release) label Aug 28, 2024
@goruha goruha changed the title Allow to get PR by ID Get the PR info by ID or associated commit Sep 5, 2024
osterman
osterman reviewed Sep 5, 2024
View reviewed changes
README.yaml Outdated Show resolved Hide resolved
@goruha @osterman
Update README.yaml
d16327b 
Co-authored-by: Erik Osterman (CEO @ Cloud Posse) <[email protected]>
@goruha goruha merged commit 04aa142 into main Sep 5, 2024
12 checks passed
@goruha goruha deleted the by-number branch September 5, 2024 16:40
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 5, 2024

These changes were released in v2.0.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@osterman osterman osterman approved these changes

Assignees
No one assigned
Labels
major Breaking changes (or first stable release)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@goruha @osterman