You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
🚀 [Breaking pre-release]: Remove AppID from GitHubApp (#523)
## Description
This pull request removes the `AppID` property from the `GitHubApp`
class and eliminates its usage throughout the codebase. The changes help
simplify the class structure and ensure consistency in object
construction.
Class structure simplification:
* Removed the `AppID` property from the `GitHubApp` class definition in
`src/classes/public/App/GitHubApp.ps1`.
* Removed assignment of `AppID` in the `GitHubApp` constructor in
`src/classes/public/App/GitHubApp.ps1`.
Object construction consistency:
* Removed the `app_id` property when creating `GitHubApp` objects in
both constructors of the `GitHubAppInstallation` class in
`src/classes/public/App/GitHubAppInstallation.ps1`.
[[1]](diffhunk://#diff-c1442a9fa3e51d2f0bf378039e97400122a24423d72d534112ca6f7c06a5ffaeL54)
[[2]](diffhunk://#diff-c1442a9fa3e51d2f0bf378039e97400122a24423d72d534112ca6f7c06a5ffaeL94)
## Type of change
<!-- Use the check-boxes [x] on the options that are relevant. -->
- [ ] 📖 [Docs]
- [ ] 🪲 [Fix]
- [ ] 🩹 [Patch]
- [ ] ⚠️ [Security fix]
- [x] 🚀 [Feature]
- [ ] 🌟 [Breaking change]
## Checklist
<!-- Use the check-boxes [x] on the options that are relevant. -->
- [x] I have performed a self-review of my own code
- [x] I have commented my code, particularly in hard-to-understand areas
0 commit comments