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

[669] Clean up code after migration #1329

Merged
merged 7 commits into from
Dec 13, 2023
Merged

[669] Clean up code after migration #1329

merged 7 commits into from
Dec 13, 2023

Conversation

saliceti
Copy link
Member

@saliceti saliceti commented Dec 11, 2023 •
edited
Loading

Description

  • Deleted paas deploment code
  • Removed references to paas
  • Reintroduced missing smoke test
  • Fixed labeler config for V5

@saliceti saliceti added the DevOps DevOps label Dec 11, 2023
@saliceti
Copy link
Member Author

GIT-API: Clean up code

@saliceti saliceti force-pushed the 669-git-api-clean-up-code branch from 706ef63 to d8319a6 Compare December 12, 2023 15:18
@saliceti saliceti force-pushed the 669-git-api-clean-up-code branch from b6d5454 to 4ce4bb7 Compare December 12, 2023 15:48
@saliceti saliceti marked this pull request as ready for review December 12, 2023 15:56
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

martyn-w
martyn-w approved these changes Dec 12, 2023
View reviewed changes
Copy link
Contributor

@martyn-w martyn-w left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks good - thanks very much for tidying this up 👍

@saliceti saliceti merged commit 58c951d into master Dec 13, 2023
12 of 13 checks passed
@saliceti saliceti deleted the 669-git-api-clean-up-code branch December 13, 2023 09:04
@martyn-w martyn-w mentioned this pull request Dec 15, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@martyn-w martyn-w martyn-w approved these changes

Assignees
No one assigned
Labels
DevOps DevOps
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@saliceti @martyn-w