Skip to content
This repository has been archived by the owner on Sep 26, 2024. It is now read-only.

Conversation

yaswanth-deriv
Copy link
Contributor

Changes:

  • Separated Docker and Kubernetes Status notification from Cloudflare Pages deployment slack notification in deriv-com repo

Type of change

  • Bug fix
  • New feature
  • Update feature
  • Refactor code
  • Translation to code
  • Translation to crowdin
  • Script configuration
  • Improve performance
  • Style only
  • Dependency update
  • Documentation update
  • Release

Copy link

vercel bot commented Jan 4, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
deriv-com ✅ Ready (Inspect) Visit Preview Jan 9, 2024 11:39am

@yaswanth-deriv yaswanth-deriv changed the title refactor: To separate docker and k8s notification [FEQ]/Yaswanth/FEQ-1089/To separate docker and k8s notification Jan 4, 2024
.github/workflows/production.yml Show resolved Hide resolved
.github/workflows/production.yml Outdated Show resolved Hide resolved
Copy link
Contributor

github-actions bot commented Jan 9, 2024

Name Result
Build status Failed ❌
Action URL Visit Action

Copy link

sonarqubecloud bot commented Jan 9, 2024

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

Copy link
Contributor

@habib-deriv habib-deriv left a comment

Choose a reason for hiding this comment

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

LGTM

@ali-hosseini-deriv ali-hosseini-deriv merged commit 40a6fa4 into binary-com:master Jan 10, 2024
5 checks passed
meenakshi-deriv pushed a commit to meenakshi-deriv/deriv-com that referenced this pull request Jan 10, 2024
…ry-com#6434)

* refactor: To separate docker and k8s notification

* refactor: To remove extra spaces

* refactor: To remove extra spaces

* refactor: added not success condition in docker slack
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants