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

release(v0.56.0): merge to prod #1050

Merged
merged 11 commits into from
Dec 6, 2023
Merged

release(v0.56.0): merge to prod #1050

merged 11 commits into from
Dec 6, 2023

Conversation

seaerchin
Copy link
Contributor

What's Changed

Full Changelog: v0.55.0...v0.56.0

kishore03109 and others added 11 commits November 16, 2023 19:19
* Added swagger doc for GIG DNS API

* Modified the swagger doc to allow 1 DNS record per POST request

* added 401 error response to all 3 API calls
Bumps [sequelize-typescript](https://github.com/RobinBuschmann/sequelize-typescript) from 2.1.5 to 2.1.6.
- [Release notes](https://github.com/RobinBuschmann/sequelize-typescript/releases)
- [Changelog](https://github.com/sequelize/sequelize-typescript/blob/master/CHANGELOG.md)
- [Commits](sequelize/sequelize-typescript@v2.1.5...v2.1.6)

---
updated-dependencies:
- dependency-name: sequelize-typescript
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
## Problem

Moving forward, since we have migrated most sites out of gh, we will only offer quickie for sites that are in ggs. 
As such, this pr removes the dependency of a no longer supported flow so as to allow for a simpler mental model. 
Additionally, there were weird abstractions bet GithubService + GithubCommitService -> removed the responsibilities to be isolated.

Test cases will be added in the leaf node of these prs, cicd will also be fixed upstream.
## Problem

Actual removal of the no longer needed `GithubCommitService` file to reduce complexity + fix test cases.
adding test cases for the two new functions `renameSinglePath` and `deleteDirectory` introduced into `GithubService` in #1042. Note that these functions already existed, but this pr adds the test cases to the right file.
## Problem

1. the presence of default arguments lead to sneaky bugs introduced due to caller unknowingly calling the wrong branch. 
2. there is a scary instance whereby the lack of update to staging-lite could occur, but the update to staging does not, leading to issues downstream. 



Closes [insert issue #]

## Solution
1. to keep things simple, we make all the args compulsory.
2. make staging update first before updating staging lite.
add test cases for `GitFileCommitService`.
* fix(rollback handler): convert to ts for safety

* Untitled commit

* Untitled commit

* Untitled commit

* Untitled commit

* fix(retry): add max 5 retries)

* feat(loggin): add logging for possibility of alarm

* feat(style): use resultAsync for clarity + refactor

* fix(privatisation): should be write handler
@seaerchin seaerchin changed the title Release/v0.56.0 release(v0.56.0): merge to prod Dec 6, 2023
@seaerchin seaerchin requested a review from a team December 6, 2023 11:08
Copy link

mergify bot commented Dec 6, 2023

⚠️ The sha of the head commit of this PR conflicts with #1049. Mergify cannot evaluate rules on this PR. ⚠️

@seaerchin
Copy link
Contributor Author

seaerchin commented Dec 6, 2023

@seaerchin #1044
@kishore03109 doing the quickie test case

@mergify mergify bot mentioned this pull request Dec 6, 2023
@seaerchin seaerchin merged commit 3ae9481 into master Dec 6, 2023
28 checks passed
@seaerchin seaerchin deleted the release/v0.56.0 branch December 6, 2023 11:28
This was referenced Jun 27, 2024
@dcshzj dcshzj mentioned this pull request Jun 27, 2024
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