Skip to content

Conversation

@coltenkrauter
Copy link
Contributor

@coltenkrauter coltenkrauter commented Mar 26, 2025

Description

#27 Add support for multiple tokens to be supplied so multiple orgs may be scanned for open PRs.
#10 Also, simplify entry point.
#3 Add tests

Screenshot 2025-03-26 at 2 07 56 PM

Type of Change

  • New feature
  • Bug fix
  • Chore
  • Configuration
  • Documentation
  • Refactoring
  • Hotfix
  • Security
  • GitFlow

Testing

  • Executed locally
  • Unit tests
  • Integration tests
  • Load tests
  • PR Status checks
  • Workflow execution (against non-default branch)
  • Other (explain)
  • Not worth it (explain)
  • Not applicable

Checklist

[Not required to be completed, just check off what you've done.]

  • Commits are reasonably sized and the commit message describes the changes well.
  • I have made corresponding changes to the documentation.
  • I have added unit tests for changes to business logic.
  • I have emitted metrics so our team can monitor the health of this system related to these changes.
  • I have added logs to ensure debugging is straight-forward when issues arise.
  • I have considered failure modes (e.g. major dependency becomes unavailable, bad data passed in).
  • I have added a comment to this PR to discuss any concerns/risks of these changes. This might include effects on performance, new dependencies, changes in behavior, or difficult things to test.
  • I have considered whether some of these changes should go into a shared package to increase code reusability.

@coltenkrauter coltenkrauter force-pushed the feature/coltenkrauter/2025-03-25/3 branch 9 times, most recently from 2095464 to f4d7927 Compare March 26, 2025 19:32
@coltenkrauter coltenkrauter force-pushed the feature/coltenkrauter/2025-03-25/3 branch from f4d7927 to 71b91f8 Compare March 26, 2025 20:09
@coltenkrauter coltenkrauter mentioned this pull request Mar 26, 2025
6 tasks
@coltenkrauter coltenkrauter force-pushed the feature/coltenkrauter/2025-03-25/3 branch 3 times, most recently from d3148e6 to 098b023 Compare March 27, 2025 20:23
@coltenkrauter coltenkrauter force-pushed the feature/coltenkrauter/2025-03-25/3 branch from 098b023 to f3cb581 Compare March 27, 2025 20:24
@atvi-tmiller
Copy link

looks good to me!

@coltenkrauter coltenkrauter self-assigned this Apr 15, 2025
@coltenkrauter coltenkrauter merged commit 761bc34 into main Apr 15, 2025
4 checks passed
@coltenkrauter coltenkrauter deleted the feature/coltenkrauter/2025-03-25/3 branch April 15, 2025 16:09
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